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

Regression: broken flameshot icon in about window ( caused by #2318 ) #2684

Closed
hosiet opened this issue Jun 18, 2022 · 3 comments · Fixed by #2693
Closed

Regression: broken flameshot icon in about window ( caused by #2318 ) #2684

hosiet opened this issue Jun 18, 2022 · 3 comments · Fixed by #2693
Assignees
Labels
Bug It's a bug

Comments

@hosiet
Copy link
Member

hosiet commented Jun 18, 2022

Flameshot Version

12.0-rc1

Installation Type

Linux, MacOS, or Windows Package manager (apt, pacman, eopkg, choco, brew, ...)

Operating System type and version

Debian Sid / Unstable

Description

The flameshot icon in about window is broken. Meanwhile, stderr has multiple output of "qt.svg: Invalid path data; path truncated.".

I believe current SVG icon has some incompatibilities with Qt5's SVG library. We all know that in #2318 the SVG icon was manually "optimized". When reverting the icon to un-optimized version (flameshot.svg and org.flameshot.Flameshot.svg from https://github.com/flameshot-org/flameshot/tree/9752b975c132056cb0fb5add4d28bf5c60ef6d84/data/img/app ), everything works well again. I am using the most up-to-date Qt5 LTS version (5.15.4) so I believe we should make things compatible with this Qt version.

Ping original author @RiedleroD so that you may look further into it.

Steps to reproduce

  1. Run /usr/bin/flameshot.
  2. Open "About" dialog from tray icon.

Screenshots or screen recordings

image

System Information

  • Debian Sid/Unstable
  • Qt 5.15.4
  • Desktop environment: (reproducible on all major Desktop Environments, including MATE and KDE Plasma)
  • X11 environment
@hosiet hosiet added the Unconfirmed Bug The bug is not confirmed by anyone else. label Jun 18, 2022
@hosiet hosiet changed the title Regression: broken flameshot icon in about window ( caused by #2138 ) Regression: broken flameshot icon in about window ( caused by #2318 ) Jun 18, 2022
@mmahmoudian
Copy link
Member

This is similar to #2667 but of course not duplicate.

@borgmanJeremy
Copy link
Contributor

I'll revert the SVG.

@borgmanJeremy borgmanJeremy added Bug It's a bug and removed Unconfirmed Bug The bug is not confirmed by anyone else. labels Jun 20, 2022
@borgmanJeremy borgmanJeremy self-assigned this Jun 20, 2022
@RiedleroD
Copy link
Contributor

RiedleroD commented Jun 20, 2022

oh, this was actually my fault (for not fully correcting for known bugs in QtSVG)

I'll open a PR with the paths fixed & you'll see if that fixes is, ok?

also I don't particularly like how you put "optimized" in quotes, but I don't want to start drama over this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug It's a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants