Skip to content

[pull] master from slgobinath:master #108

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

Merged
merged 5 commits into from
May 4, 2025
Merged

Conversation

pull[bot]
Copy link

@pull pull bot commented May 4, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

deltragon added 5 commits May 4, 2025 13:27
Previously, the stylesheet was copied to the user's .config dir, and
this was then loaded as a style (with priority "application").
The user could alter this stylesheet directly.
This stylesheet was called "safeeyes_styles.css".

Now, we split the stylesheets into two:
- The system stylesheet, which is directly loaded with priority
  "application".
- The user's stylesheet, if it exists, which is loaded from the .config
  dir with priority "user", overriding both the system stylesheet and
  any themes the user might have.
  This is now called "safeeyes_custom_styles.css".

This means in the future, we can simply add new default styles to the
system stylesheet without breaking the user. Additionally, any styles
loaded with a higher priority than "application" (eg. certain GTK
themes, see #687), can now
be overridden by the user's stylesheet.

We also check if the old path ("safeeyes_styles.css") contains an old
default stylesheet. If it does, we delete it.
If it (likely) contains a customized stylesheet, don't delete it, but
warn the user that they should use "safeeyes_custom_styles.css" instead
now.
rework custom stylesheet loading
@pull pull bot added the ⤵️ pull label May 4, 2025
@pull pull bot merged commit ef1cfca into Uncodedtech:master May 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant