Bug 442993

Summary: ktimetracker icon is much smaller than viewport for 22x22 size
Product: [Plasma] Breeze Reporter: Alexander Potashev <aspotashev>
Component: IconsAssignee: visual-design
Status: RESOLVED FIXED    
Severity: normal CC: kainz.a, nate, uhhadd
Priority: NOR    
Version: git-master   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In: 5.87
Bug Depends on:    
Bug Blocks: 442996    
Attachments: screenshot (see first row)

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