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

Improve scaling change message to request further user action. #120

Merged
merged 2 commits into from May 13, 2022

Conversation

fossfreedom
Copy link
Contributor

@fossfreedom fossfreedom commented May 9, 2022

Description

Testing of various scaling changes across different monitor and graphic driver types shows that budgie desktop popovers and other elements of the GUI do not always refresh correctly. As a quick win lets prompt the user to restart their session to allow the desktop to refresh & resize properly.

Screenshot from 2022-05-09 10-01-34

As an aside there doesn't appear to be any way to force mutter based dialogs to ensure messages span over more than one button. Odd - just odd.

As discussed on matrix/development with @EbonJaeger - but tried to further reduce the message size to improve appearance without affecting potential translation understanding.

Submitter Checklist

  • Squashed commits with git rebase -i (if needed)
  • Built budgie-desktop and verified that the patch worked (if needed)

Testing of various scaling changes across different monitor and
graphic driver types shows that budgie desktop popovers and other
elements of the GUI do not always refresh correctly.  As a quick
win lets prompt the user to restart their session to allow
the desktop to refresh & resize properly.
@JoshStrobl
Copy link
Member

Testing of various scaling changes across different monitor and
graphic driver types shows that budgie desktop popovers and other
elements of the GUI do not always refresh correctly. As a quick
win lets prompt the user to restart their session to allow
the desktop to refresh & resize properly.

80-col bad, please fix. That being said, I can't land this with it looking that way. Super bork.

Copy link
Member

@JoshStrobl JoshStrobl left a comment

Choose a reason for hiding this comment

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

🛑 (GitHub wanted a comment)

@fossfreedom
Copy link
Contributor Author

Screenshot from 2022-05-09 11-57-49

@JoshStrobl
Copy link
Member

LGTM. Went ahead and updated the description since it was hard wrapping after 80 columns. PR itself looks good. Thanks!

@JoshStrobl JoshStrobl merged commit 56fde8e into master May 13, 2022
@JoshStrobl JoshStrobl deleted the resolution branch May 13, 2022 22:48
JoshStrobl pushed a commit that referenced this pull request Aug 21, 2022
* Improve scaling change message to request further user action.
Testing of various scaling changes across different monitor and graphic driver types shows that budgie desktop popovers and other elements of the GUI do not always refresh correctly.  As a quick win lets prompt the user to restart their session to allow the desktop to refresh & resize properly.

* Write our own version of Meta.Util.show_dialog
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.

None yet

3 participants