Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: replace all deprecated elements #55

Merged
merged 2 commits into from
Jun 3, 2018
Merged

UI: replace all deprecated elements #55

merged 2 commits into from
Jun 3, 2018

Conversation

raveit65
Copy link
Member

Please test.

@raveit65 raveit65 requested a review from a team May 31, 2018 16:42
@raveit65 raveit65 changed the title UI: drop all deprecated elements UI: replace all deprecated elements May 31, 2018
Copy link
Member

@lukefromdc lukefromdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not find any noticeable differences in appearance or behavior with a quick test (open mate-calc, punch in some numbers, test keys known to pop up menus). Did not test calculations for accuracy but this should not be affected.

@vkareh vkareh requested a review from a team May 31, 2018 19:04
Copy link
Member

@vkareh vkareh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well. I opened all these files in Glade and I could not see any warnings whatsoever - great work! :)

@vkareh vkareh requested a review from a team May 31, 2018 19:05
@sc0w
Copy link
Member

sc0w commented Jun 1, 2018

There is one more deprecation, stock items, glade never shows us it, I reported it in the past:

https://gitlab.gnome.org/GNOME/glade/issues/289

@raveit65 I can add one more commit here to fix it if you agree :)

@raveit65
Copy link
Member Author

raveit65 commented Jun 1, 2018

@sc0w
Oh i didn't know that. Of course i agree with adding commits here :)

@sc0w
Copy link
Member

sc0w commented Jun 1, 2018

ok, done

Copy link
Member

@vkareh vkareh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still works well with the new change. Thanks!

@vkareh vkareh requested a review from a team June 1, 2018 21:21
@raveit65
Copy link
Member Author

raveit65 commented Jun 2, 2018

Hmm, last commit breaks the dialogs in eg. financial mode.
I got this warning

[rave@mother ~]$ mate-calc

** (mate-calc:6710): WARNING **: 09:57:48.701: Error loading button UI: /usr/share/mate-calc/buttons-financial.ui:37:41 Object with ID image1 not found

and 'cancel buttons' doesn't have a icon any more.
Also, it seems that the buttons lost functions. Try to use the rate dialog.
Before:
mate-calc-rate-dialog
After:
mate-calc-rate-dialog-broken

@sc0w
Copy link
Member

sc0w commented Jun 2, 2018

I did some changes

@vkareh @lukefromdc @raveit65 please test again

@lukefromdc
Copy link
Member

The cancel button is back, but somewhere in this we've picked up some out of position combo box menus. Image coming

@lukefromdc
Copy link
Member

lukefromdc commented Jun 2, 2018

mate-calc-comboboxes_6-2-2018

This happened sporadically, not all the time so when it happened I had to use a camera rather than a screenshot to capture it. In case you can't tell, the button reading "Degrees" is the active one the menu is supposed to be attached to

@raveit65
Copy link
Member Author

raveit65 commented Jun 3, 2018

I can reproduce this with master too, so it's not related to UI changes here.
You need to open the comboboxes in financial mode first , than try to open the comboboxes in advanced mode. A little video which demonstrate it. (5.7 MB)
https://www.dropbox.com/s/jf8axx21u4xlhi3/mate-calc.ogv.tar.xz?dl=0
Can you please file another report for this new issue?

@lukefromdc
Copy link
Member

I too just confirmed this with master, so this PR should be OK. New issue reported at
#56

@raveit65
Copy link
Member Author

raveit65 commented Jun 3, 2018

Icons and function for buttons are back with latest change.

@raveit65 raveit65 merged commit 58f106a into master Jun 3, 2018
@raveit65 raveit65 deleted the UI-deprecations branch June 3, 2018 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants