Bug 442993 - ktimetracker icon is much smaller than viewport for 22x22 size
Summary: ktimetracker icon is much smaller than viewport for 22x22 size
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: Icons (show other bugs)
Version: git-master
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: visual-design
URL:
Keywords:
Depends on:
Blocks: 442996
  Show dependency treegraph
 
Reported: 2021-09-26 19:17 UTC by Alexander Potashev
Modified: 2021-10-04 13:58 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.87


Attachments
screenshot (see first row) (107.88 KB, image/png)
2021-09-26 19:17 UTC, Alexander Potashev
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Potashev 2021-09-26 19:17:40 UTC
Created attachment 141933 [details]
screenshot (see first row)

SUMMARY
Actual dimensions of content in ./icons/apps/22/ktimetracker.svg are smaller than image size detected by image viewers, making this icon in the UI look much smaller than it should be (see screenshot.)

Please verify all related icons:
./icons/apps/32/ktimetracker.svg
./icons/apps/48/ktimetracker.svg
./icons/apps/22/ktimetracker.svg
./icons-dark/apps/32/ktimetracker.svg
./icons-dark/apps/48/ktimetracker.svg
./icons-dark/apps/22/ktimetracker.svg

The icons were originally created in https://phabricator.kde.org/D26441

STEPS TO REPRODUCE
1. 
2. 
3. 

OBSERVED RESULT


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2021-09-27 16:02:25 UTC
The 32px version looks okay to me in Cuttlefish, but I can see the issue with the 22px icon there.
Comment 2 Nate Graham 2021-09-27 16:17:37 UTC
Quick fix not possible as there are multiple issues including non-pixel alignment. the icon probably needs to be re-done.
Comment 3 Bug Janitor Service 2021-09-28 19:08:50 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/128
Comment 4 Manuel de la Fuente 2021-09-28 22:16:10 UTC
Git commit f0c1e0cac0e9f8b2a4126ace73e94ac4fa00949b by Manuel Jesús de la Fuente.
Committed on 28/09/2021 at 22:16.
Pushed by mdelafuente into branch 'master'.

Fix KTimeTracker icon sizes

M  +1    -39   icons-dark/apps/22/ktimetracker.svg
M  +1    -39   icons-dark/apps/32/ktimetracker.svg
M  +1    -39   icons-dark/apps/48/ktimetracker.svg
M  +1    -39   icons/apps/22/ktimetracker.svg
M  +1    -39   icons/apps/32/ktimetracker.svg
M  +1    -39   icons/apps/48/ktimetracker.svg

https://invent.kde.org/frameworks/breeze-icons/commit/f0c1e0cac0e9f8b2a4126ace73e94ac4fa00949b