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

Icon fixes #715

Merged
merged 6 commits into from Jun 21, 2022
Merged

Icon fixes #715

merged 6 commits into from Jun 21, 2022

Conversation

corubba
Copy link
Contributor

@corubba corubba commented Mar 12, 2022

Parts of this PR were originally in #687.

Changes

Clean up icons

Remove those that are not actually used anywhere. Some of them are broken to begin with (size of 0 byte).

Fix action aliases

For some reason caseconversion.action and standard.action had crossed-over aliases, but since they are always used together it never broke things.

Fix appicon

There is a vector version, use that instead of keeping a separate png around. Since I found the appicon name more descriptive, use that consistently.

Update puddlelogo

  • Moved the svg from the root to the rest of the images
  • Resized it from weird 464x127 to not-as-much-weird 512x128 pixel
  • Named the layers and objects more descriptively
  • Removed the transparency from the background gradient while keeping
    the resulting colours the same. Also resized it to fit the new size,
    and made the white border consistently 2px all around
  • The font used for the app name is Impossibilium BRK by Ænigma Fonts
    (Brian Kent). It can be found freely on the internet, for example
    here
  • Created a path copy so you don't need the font installed
  • Re-centered the text
  • Updated, repositioned and hid the version text (since it will be
    outdated soon anyway)
  • Re-exported the svg to png

In theory we could use the svg directly, but the svg renderer in Qt does not support filters, which are used to create the backdrop shadow. And since it looks ugly/weird without it, we continue to use a png version.

Recompile resource.py

$ pyrcc5 puddlestuff/resource.qrc -o puddlestuff/resource.py

Add dynamic version to splashscreen

The version was removed from the png, so add it back in on the fly during runtime with the actual version.

Comparison Splashscreen

Before

before

After

after

Remove those that are not actually used anywhere. Some of them are
broken to begin with (size of 0 byte).
For some reason `caseconversion.action` and `standard.action` had
crossed-over aliases, but since they are always used together it
never broke things.
There is a vector version, use that instead of keeping a separate png
around. Since I found the `appicon` name more descriptive, use that
consistently.
- Moved the svg from the root to the rest of the images
- Resized it from weird 464x127 to not-as-much-weird 512x128 pixel
- Named the layers and objects more descriptively
- Removed the transparency from the background gradient while keeping
  the resulting colours the same. Also resized it to fit the new size,
  and made the white border consistently 2px all around
- The font used for the app name is `Impossibilium BRK` by Ænigma Fonts
  (Brian Kent). It can be found freely on the internet, for example
  here [0]
- Created a path copy so you don't need the font installed
- Re-centered the text
- Updated, repositioned and hid the version text (since it will be
  outdated soon anyway)
- Re-exported the svg to png

In theory we could use the svg directly, but the svg renderer in Qt does
not support filters [1], which are used to create the backdrop shadow.
And since it looks ugly/weird without it, we continue to use a png
version.

[0] https://www.1001fonts.com/impossibilium-brk-font.html
[1] https://bugreports.qt.io/browse/QTBUG-9081
$ pyrcc5 puddlestuff/resource.qrc -o puddlestuff/resource.py
The version was removed from the png, so add it back in on the fly
during runtime with the actual version.
@corubba corubba added this to the PyQt6 milestone Mar 12, 2022
@sandrotosi sandrotosi merged commit 720072a into puddletag:master Jun 21, 2022
@sandrotosi
Copy link
Member

thanks!

@corubba corubba deleted the bugfix/icons branch June 21, 2022 19:14
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

2 participants