<table><tr><td style="">ndavis edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-b4uwmdrjwnrfwvb/">(Show Details)</a>
</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/D17623">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>There is currently an issue where there is no good folder to place these. `apps/32` is a `Fixed` directory, which means this patch will fail the scalable test. `preferences/32` would work, except it's only meant to be used for icons that are used by the KDE System Settings. Currently, most of these icons are 32px because that's the size they will be at in the YaST Control Center. Another way to solve the scalable issue would be to also create 48px versions of all of these icons<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">. However</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">but </span>that would be a lot of work and I want these icons to be in KF5.54 so that openSUSE Leap 15.1 will have them.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17623">https://phabricator.kde.org/D17623</a></div></div><br /><div><strong>To: </strong>ndavis, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>