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

Light/Dark theme for ublue-motd acting funky #2017

Closed
tulilirockz opened this issue Dec 6, 2024 · 9 comments
Closed

Light/Dark theme for ublue-motd acting funky #2017

tulilirockz opened this issue Dec 6, 2024 · 9 comments
Labels
bug Something isn't working

Comments

@tulilirockz
Copy link
Collaborator

The light/dark detection method used by ublue-motd is not working too good on environments where gsettings is misreporting light/dark color schemes, like ssh sessions, distroboxes and so on. It also does not detect it properly on KDE (but given that Aurora is splitting that is not as much of a problem anymore).

I believe that by sending a patch upstream that allows configurable light/dark theme selection this might be resolved, but for now, what do you guys think should be done?

I believe we should just disable it for now or make it force dark mode

@fiftydinar
Copy link
Collaborator

fiftydinar commented Dec 6, 2024

I already proposed my solution for this:

# Default fallback theme color
THEME="'slate'"

# Accent theme color if available
if command -v gsettings > /dev/null; then
 if gsettings get org.gnome.desktop.interface accent-color > /dev/null 2>&1; then
   THEME="$(gsettings get org.gnome.desktop.interface accent-color)"
 fi
fi

You can add some more robust checks additionally to this if you wish

@bsherman
Copy link
Contributor

bsherman commented Dec 7, 2024

I'm not sure if it's light/dark related, but I was noticing that the motd matching the accent color of GNOME... it feels a bit overdone...

The default (slate? ) is the only one which doesn't go quite as overboard, maybe?

Image
Image
Image
Image

@castrojo
Copy link
Member

castrojo commented Dec 7, 2024

Ok now that I've played with it a bit I agree it's a bit over the top. I would color the top line, because that looks good, then color the vertical/horizontal bar cross thing, and then the icons for the bullet points. And that's it.

Then leave everything else white, including the ujust section. Monochrome with the highlight box will be fine for the example commands, no need for color there.

@tulilirockz
Copy link
Collaborator Author

What you guys think about setting a lighter gray for the code_highlight box and using the same color for white and dark theme? - I believe that would look nice and fix the funkiness about having either a light or dark theme

@castrojo
Copy link
Member

castrojo commented Dec 7, 2024

Sure, whatever looks awesome, dealer's choice!

@tulilirockz
Copy link
Collaborator Author

But I do agree that the accent colors on everything is quite over the top LOL - I LOVED it at first but as jorge said, after playing around with it a bit I believe it is a bit much indeed

@tulilirockz
Copy link
Collaborator Author

Hello everyone! After a bit of discussion and testing, we ended up changing the theme to this and it should work good on light/dark color schemes now since there wont be such a big difference!

Image
Image

@mfkp
Copy link

mfkp commented Jan 14, 2025

After pulling in the latest updates, now I'm getting this error upon opening any new terminals:

Error: Specified style does not exist: /usr/share/ublue-os/motd/themes/\slate\.json

Is it related to this change?

@tulilirockz
Copy link
Collaborator Author

Nope! This got fixed recently, after updating you should get this fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants