Review Request 129807: proposal: add ksvg2ico

René J.V. Bertin rjvbertin at gmail.com
Thu Jan 12 16:47:43 UTC 2017


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

(Updated Jan. 12, 2017, 5:47 p.m.)


Review request for KDE Frameworks, kdewin, Alex Merry, and Christoph Feck.


Changes
-------

certain ico viewers show the 256x256 8-bit icon instead of its ARGB32 version; this is the case of RR's attachment preview but not of the full viewer which is open while I'm typing this.


Repository: kiconthemes


Description
-------

Recently I committed a few changes to `ecm_add_app_icon` that make use of the fact that KIconThemes includes a `ksvg2icns` utility, making it possible to generate an application icon from an SVG file instead of from a series of PNG files. 

With the present patch I propose to add an updated and partly rewritten version of the ageing `svg2ico` tool from the KDEWin project to KIconThemes. The goal is to have a proper and up-to-date tool to generate application icons from SVG on MS Windows too.

I considered proposing it to KDEWin instead, but in the end I liked the idea better to provide icon conversion utilities for Mac and Windows with a framework, where they might also serve in a less KDE-centric context (and cross-platform development).
I also played with the idea of integrating the png2ico step into the tool, but noticed during testing that Matthias Benkmann's png2ico supports input sizes up to 248x248 while KDEWin's png2ico is currently limited at 48x48 or maybe even 32x32. Lacking a MSWin development system I'm not volunteering to update that code.


Diffs (updated)
-----

  src/CMakeLists.txt efba9e6 
  src/tools/ksvg2ico/CMakeLists.txt PRE-CREATION 
  src/tools/ksvg2ico/ksvg2ico.cpp PRE-CREATION 
  src/tools/ksvg2ico/qicohandler.h PRE-CREATION 
  src/tools/ksvg2ico/qicohandler.cpp PRE-CREATION 

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


Testing
-------

On Mac OS X 10.9 with Qt 5.7.1 and KF5 5.29.0 .


File Attachments (updated)
----------------

.ico file generated from sc-apps-calligrakarbon.svgz using ksvg2ico with Benkmann's png2ico
  https://git.reviewboard.kde.org/media/uploaded/files/2017/01/11/aa0973dc-1be8-4d36-82dc-8f7baa053fee__apps-calligrakarbon.ico
.ico created by icotool from *-apps-calligrakarbon.png
  https://git.reviewboard.kde.org/media/uploaded/files/2017/01/12/3bbee1f5-6770-4d6e-847d-e32c7e5d3fe1__icotool-apps-calligrakarbon.png
ico file generated by the standalone ksvg2ico
  https://git.reviewboard.kde.org/media/uploaded/files/2017/01/12/7b286997-3100-47d0-800e-acb56ee9c68f__apps-calligrakarbon.ico


Thanks,

René J.V. Bertin

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


More information about the Kde-frameworks-devel mailing list