<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D14971">View Revision</a></tr></table><br /><div><div><p>If we rename the icons, we will need to correspondingly change the OSD code itself to use the new names: <a href="https://cgit.kde.org/kscreen.git/tree/kded/qml/OsdSelector.qml#n51" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/kscreen.git/tree/kded/qml/OsdSelector.qml#n51</a></p>

<p>Unfortunately, that code lives in <tt style="background: #ebebeb; font-size: 13px;">KScreen</tt>, which uses the Plasma release schedule rather than the Frameworks schedule. Therefore we cannot guarantee that people will be using the right combination of the two pieces of code; it's perfectly possible for a distro to ship KDE Frameworks 5.50 (with the renamed files) but KDE Plasma 5.13.5 (which would use the old names). If these lived on-disk, I'd suggest just making a symlink. Can we do that here internally? Is there any way to give one image two names in the svgz file?</p>

<p>Either way, that should be done in a separate commit. Please revert the name change and we'll do that later. It's important to keep patches as small as possible to keep the git history clean, and also make it easy to revert any individual change that caused problems.</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/D14971">https://phabricator.kde.org/D14971</a></div></div><br /><div><strong>To: </strong>pstefan, ngraham, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>