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

[HELP WANTED] Fix for System Menu Styles for Gnome 48 #517

Merged
merged 4 commits into from
Mar 27, 2025

Conversation

marmitar
Copy link
Contributor

@marmitar marmitar commented Mar 22, 2025

For a clean diff, see marmitar/Orchis-theme#1

This is a fix for #514 and #516. The fix does work overall, but it has some coloring issues.

Any opinion or help here would be appreciated. You can find all changes for Gnome 48 themes on this diff. The important file is src/_sass/gnome-shell/widgets/_quick-settings-48.scss. My changes are on the last two wip commits.

Visual Reference

Here's the base fix:

Screenshot From 2025-03-22 20-02-48

Notice that new separator there? I didn't know which color to use, so I removed it:

Screenshot From 2025-03-22 20-05-37

Here you can see that the arrow has a different background color from the button when unchecked, but the same color when checked. Since I don't know the colors used before, I decide to just use the button colors for everything. Here's how it looks:

Screenshot From 2025-03-22 20-06-55

So that's the current state of this fix.

@marmitar marmitar changed the title [WIP] Fix for System Menu Styles [HELP WANTED] Fix for System Menu Styles Mar 23, 2025
@marmitar marmitar changed the title [HELP WANTED] Fix for System Menu Styles [HELP WANTED] Fix for System Menu Styles for Gnome 48 Mar 23, 2025
@jrangelardila
Copy link

Hello @marmitar

Your changes, worked very well in Arch with gnome 48.

Thanks!

@masterdokuro
Copy link

Thanks for taking the time to look at this. However, this fix changes the blue colour of the part of the button which has the -> arrow. You can see in the screenshot I included in #514 that right hand side of the button has a lighter blue to the rest of the button. I think in the spirit of this theme we would want to keep that styling with any fix being made to gnome 48 so it matches what we have today with gnome 47.

Again, thanks for looking into this.

@marmitar
Copy link
Contributor Author

marmitar commented Mar 23, 2025

Hi, @masterdokuro, thanks for taking a look at this.

Yes, the color needs to be fixed, but I'm not sure which color to use there. It wasn't defined in quick-toggle-arrow before, so it was probably selected by another CSS class. This change was implemented on gnome-shell!3554, I'll take a look there to see if I can find out how it was styled before.

About the icon, I don't think we can change it. It's defined as go-next-symbolic even before Gnome 47 (see quickSettings.js). That icon is a chevron on both Adwaita and Tela Circle (the one I'm using and developed by @vinceliuice), so I'm not sure how you got an arrow there. Are you maybe using another icon theme for that?

@masterdokuro
Copy link

masterdokuro commented Mar 23, 2025

The icon itself comes from the icon theme I use (papirus), so ignore that. Sorry if I confused you mentioning the icon, its not important to this issue. Its the colour of that part of the button which is important.

I'll take a look around and see if I can figure out what colour it is.

@vinceliuice vinceliuice merged commit e5b2d60 into vinceliuice:master Mar 27, 2025
1 check passed
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

4 participants