Review Request 126740: Add a script for optimizing svgs
andreas kainz
kainz.a at gmail.com
Thu Jan 14 12:22:32 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126740/#review91064
-----------------------------------------------------------
I have no problems to optimzie the svg files BUT my problem with the svg files is that they become more complex all the time last time the stylesheet, now optimization, ....
I love scripts BUT it would be awesome for us to have this scripts before an distribution build there packages. In git keep it as simple as possible and when an distro will use the icons use therefore an script (check if the stylesheets fit, compress, make png files, ....)
- andreas kainz
On Jan. 14, 2016, 3 vorm., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126740/
> -----------------------------------------------------------
>
> (Updated Jan. 14, 2016, 3 vorm.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Repository: breeze-icons
>
>
> Description
> -------
>
> Dirk's review (https://git.reviewboard.kde.org/r/126738/) gave me the idea that right now we're serving right away the svg's from inkscape and there's room for improvement, potentially.
>
> This patch just introduces a script that optimizes the svg's using `svgo`.
>
> More could be done, like using gzip files, we can look into that if anyone's interested. In fact, we used to use svgz for the icons, I wonder why that changed.
>
> This will change the files in-place rather than as a build step, which is what I considered first. The process to run svgo on every file was about 30 minutes to 1h on my system, so I doubt it's really desirable.
>
> A reduced file size is important because it will greatly reduce disk IO, which is a bottle-neck we have.
>
>
> Diffs
> -----
>
> optimize-svg.sh PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/126740/diff/
>
>
> Testing
> -------
>
> ```
> kde-devel at oliver:~/frameworks/breeze-icons (master)$ du -sh icons icons-dark/
> 32M icons
> 32M icons-dark/
>
> #run the script
>
> kde-devel at oliver:~/frameworks/breeze-icons (master)$ du -sh icons icons-dark/
> 17M icons
> 17M icons-dark/
> ```
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160114/bae653b3/attachment.html>
More information about the Kde-frameworks-devel
mailing list