-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: use a paned look #1621
feat: use a paned look #1621
Conversation
- to reduce the number of visual color background areas, use the paned look that can be found in gtk4 apps - since the right titlebar then has more contrast against the search field, remove the searchfield fill color override, this also... Fixes #1612
CC @anasereijo |
@anasereijo |
@Feichtmeier sorry for the delayed reply. We will be revisiting the App Center next cycle (starting mid May). These are great proposals, and we will be looking at this for sure when we start looking at the store again. Thanks for your input 🙏 |
Just had another look on this, and I think the settings button is fine in the left panel. The big improvement (imo) with this change is the visual center. Because now the right pane content and headerbar, including the searchbar, are centered around the same axis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, can you open an issue for moving the other things around @Feichtmeier ?
EDIT: I opened #1725
@Feichtmeier changes approved, it looks much better! Thanks for doing this, and apologies for taking a long time to come back to you. |
Fixes #1612
Fixes #1562