Review Request: Do not fallback to Hicolor when Oxygen provides a "-x-generic" version of an icon

Aurélien Gâteau agateau at kde.org
Mon Jun 21 22:56:26 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4403/
-----------------------------------------------------------

(Updated 2010-06-21 21:56:26.015670)


Review request for kdelibs and Rafael Fernández López.


Changes
-------

Only replace '/' with '-' when loading mime type icons.


Summary
-------

When loading an icon for video-mp4, KIconLoader incorrectly returns "video.png" from Hicolor, instead of returning "video-x-generic.png" from Oxygen. Since Hicolor only has a 16x16 version, this results in blurry icons in Dolphin or Gwenview when icons are bigger than 16x16.

This is because the current code fallbacks to Hicolor before looking for a "-x-generic" version.


Diffs (updated)
-----

  trunk/KDE/kdelibs/kdeui/icons/kiconloader.cpp 1140966 
  trunk/KDE/kdelibs/kdeui/tests/kiconloader_unittest.cpp 1140966 

Diff: http://reviewboard.kde.org/r/4403/diff


Testing
-------

- Added a unit-test entry to check we get video-x-generic.
- Icons are no longer blurry after the patch.


Thanks,

Aurélien

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100621/fc53a3df/attachment.htm>


More information about the kde-core-devel mailing list