----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126740/ -----------------------------------------------------------
(Updated Jan. 17, 2016, 9:15 p.m.) Review request for KDE Frameworks and Plasma. Changes ------- Update script to work with svgz files: to 3.3M to 3.0M in plasma-framework (plus the advantages it might have to run simpler svg). 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 (updated) ----- optimize.svg.sh PRE-CREATION Diff: https://git.reviewboard.kde.org/r/126740/diff/ Testing ------- ``` kde-devel@oliver:~/frameworks/breeze-icons (master)$ du -sh icons icons-dark/ 32M icons 32M icons-dark/ #run the script kde-devel@oliver:~/frameworks/breeze-icons (master)$ du -sh icons icons-dark/ 17M icons 17M icons-dark/ ``` Thanks, Aleix Pol Gonzalez
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel