<table><tr><td style="">kossebau added a comment.
</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><div><p>Recent example where plain SVGs in the repo would have made the change immdiate visible, as only the strings of some id attributes where changed (to match the new icon names), but instead all we see with our tool is an exchange of the complete binary blob:  <a href="https://phabricator.kde.org/D20261" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20261</a></p>

<p>So, do I correctly see that people are rather positive about this change, at least no-one objecting it? If so, I would propose to merge it after KDE Frameworks 5.57 has been branched this WE, i.e. merge Friday next week for another 7 days of consideration, but with some weeks before the next release so people using master can test-drive things some more before release, not only me :)</p>

<p>Who feels like giving  explicit +1 or the needed Accepts? :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</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>To: </strong>kossebau, Plasma, VDG<br /><strong>Cc: </strong>bruns, GB_2, ndavis, ngraham, fvogt, kde-frameworks-devel, michaelh<br /></div>