Review Request 126199: Don't warn when SVG(Z) icons are provided with multiple sizes/levels of detail

Jarosław Staniek staniek at kde.org
Sun Nov 29 20:59:29 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126199/
-----------------------------------------------------------

Review request for KDE Frameworks and Alex Merry.


Repository: extra-cmake-modules


Description
-------

This change adds SVG(Z) extensions to the list of allowed icons for specific sizes.

This technically works and is practiced, e.g. for some Breeze icons.

Reasoning:
Some SVG icons do not scale well down from 32 to 22 or up from 16 to 22.
For such cases icons are typically specially crafted for 22 and 16, at least.
Then there's no single icon that be marked as "sc".

So warnings such as:

CMake Warning at ..../ECMInstallIcons.cmake:272 (message):
    Fixed-size icon foo.svg is not PNG or MNG

... are misleading.


Diffs
-----

  modules/ECMInstallIcons.cmake 549ebe1 

Diff: https://git.reviewboard.kde.org/r/126199/diff/


Testing
-------

ECM no longer warns for projects that use .svg icons of multiple variants (e.g. kreport.git master)


Thanks,

Jarosław Staniek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151129/da347e10/attachment.html>


More information about the Kde-frameworks-devel mailing list