<table><tr><td style="">kossebau created this revision.<br />kossebau added reviewers: Plasma, VDG.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />kossebau requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20166">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The SVG format being based on plain text, storing the SVG in the repository<br />
not as .svgz, but .svg, helps both VCS tools (patching, showing diffs) as<br />
well as allows some developers to edit the SVG directly in any text editor,<br />
not only those which support automatic conversion from/to gzip format.</p>

<p>While most artists will continue (and which shall be okay) to use GUI<br />
editors like inkscape, which might rewrite the complete structure on saving,<br />
using uncompressed format in the repo still allows the occasional direct<br />
edit of the text, .e.g. to change a colour, which then is also easily seen<br />
in the commit diff.</p>

<p>To still keep the svgz format when deployed, a build step is introduced,<br />
which uses gzip to create svgz files in the build dir. This conversion can<br />
be controlled using the option GZIP_DESKTOPTHEME_SVG (default: ON).</p>

<p>With such a build processing in place for SVG files, next to that an<br />
optional stripping of the installed SVG files is added, which removes data<br />
which is not needed for rendering, e.g. indentation whitespaces or<br />
editor-only XML data. This experimental stripping can be controlled using<br />
the option STRIP_DESKTOPTHEME_SVG (default: OFF).</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Themes are still working (with & without cache removed).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>uncompressedsvginrepo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20166">https://phabricator.kde.org/D20166</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
cmake/Findgzip.cmake<br />
cmake/Findsvgcleaner.cmake<br />
src/desktoptheme/CMakeLists.txt<br />
src/desktoptheme/air/CMakeLists.txt<br />
src/desktoptheme/breeze/CMakeLists.txt<br />
src/desktoptheme/oxygen/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>kossebau, Plasma, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>