Review Request 126740: Add a script for optimizing svgs

[andreas kainz]

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



there was also an discussion to use png files cause they are smaler in size but than I thing the stylesheet stuff didn't work. I have no problem to change to .svgz the only problem is that we have a lot linked files and the links will be broken when we change the extension so please include this issue into your script.
My decission would to to change to svgz and run the svg optimizing script ones in future by each change the dev can say run script if wanted. BUT as we already have this stylesheet script (marco) please make one script and say me how it works.

- andreas kainz


On Jan. 19, 2016, 10:54 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126740/
> -----------------------------------------------------------
> 
> (Updated Jan. 19, 2016, 10:54 p.m.)
> 
> 
> 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/plasma-devel/attachments/20160119/81b351a0/attachment.html>