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

Clock Applet: Add scrollbar to Locations lists when the lists get too long #1453

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thesquash
Copy link
Member

Before this pull request, the Clock Applet's popup window would expand off the screen if the user added more than a few locations to the Locations list. In addition, the list of locations in the Clock Applet Preferences could get extremely tall as well, and could even push the bottom part of the Preferences dialog off the screen.

This PR adds (vertical) scrollbars to both widgets. Here's how the Clock Applet popup looks now:
mate-panel-clock-applet-locations-with-scrollbar

And here is how the Preferences dialog looks now:
mate-panel-clock-applet-preferences-locations-with-scrollbar

… long

Before this commit, the Clock Applet's popup window would expand off the
screen if the user added more than a few locations to the Locations list.
In addition, the list of locations in the Clock Applet Preferences could get
extremely tall as well, and could even push the bottom part of the
Preferences dialog off the screen.

This commit adds (vertical) scrollbars to both widgets.
@lukefromdc
Copy link
Member

Issue confirmed here

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.

The scrollbar works, the total size of the calendar window still pushes about half of the top bar (where the "edit" button is found) off the screen
Top_Right_of_Screen_10-15-2024

@lukefromdc lukefromdc requested a review from a team October 15, 2024 19:10
@lukefromdc
Copy link
Member

Any updates on this? It would be good to fix the issue of cutting off half the top bar when the list is too long, but at least this ensures nobody loses access to the edit button and finds themselves unable to restore access to it. We do in fact need this one;

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.

2 participants