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

e621 style tweaks #1647

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

ScoreUnder
Copy link

This PR:

  • Tweaks default colours in e621 stylesheet to use the new qproperty stuff as intended (not just the darkmode defaults that were put in there as a placeholder)
  • Adds a new stylesheet which goes a bit further in trying to mimic the style of the site
  • Contains some small widget-painting changes that enable the background of certain custom widgets to be transparent and/or selected by QSS.

That list of changes also roughly corresponds to the three commits contained in this PR, so the scarier one with code changes can be left out if need be.


Original e621 stylesheet if you haven't added default palette colours
image

Tweaked e621 stylesheet (just putting recommended defaults into the right place)
image

New 'reduxed' stylesheet (I haven't replaced the old one because I expect this to be more fragile in its design)
image

Tag manager pre-'redux':
image

Tag manager post-'redux':
image

If you leave out the changes to how transparency works, everything looks a little flatter and the rounded corners on the top right and bottom right of the Page don't draw correctly, but otherwise it's mostly fine:
image

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.

1 participant