<table><tr><td style="">ngraham 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/D13064">View Revision</a></tr></table><br /><div><div><p>I don't object to your proposal per se, but I don't think it goes far enough to address the need here. Let me give you a specific example I'm familiar with:</p>

<p>I recently submitted a patch (<a href="https://phabricator.kde.org/D13026" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D13026</a>) to fix a functional problem with Kate's <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Schema</span></span></span> toolbar menu button (<a href="https://bugs.kde.org/show_bug.cgi?id=353747" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=353747</a>). Because of the bug that Hugo's patch here fixes, my Kate patch has the effect of removing the downward-pointing arrow, so the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Schema</span></span></span> button in Kate's toolbar will just be a single word, with no icon, and no downward-pointing arrow. It'll be just a piece of text awkwardly sitting there in the toolbar. In other words, without Hugo's patch here, fixing the functional problem (inappropriate requirement to click-and-hold) will cause a visual regression (the existing downward-pointing arrow disappears).</p>

<p>We can of course later give the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Schema</span></span></span> an icon that you or someone else creates, but that's work for you (or Andreas, or someone else), and work for someone else to patch Kate to use the icon. Multiply this by the number of toolbar buttons affected by the issue throughout KDE software. We're talking about potentially a lot of icon design tasks and patches for apps to use them. And this burdens 3rd-party icon themes by increasing the number of icons they need to provide that will only be useful for users of the Breeze widget theme. And then what happens for people who use Breeze Icons or a compliant icon theme with a different widget theme that <em>does</em> draw downward-pointing arrows for menu buttons in toolbars? You'll have two downward-pointing arrows. Seems messy.</p>

<p>Compare that complication and workload to just accepting this patch, which gives them all downward-pointing arrows in Breeze and resolves the issue without any icon design work or patches for individual apps or conflicts with other themes or extra work for 3rd-party icon themes. So to me it makes the most sense to handle this here in the theme rather than with an icon.</p>

<p>If the objection to the arrow is that it's ugly, maybe we could come up with a prettier one?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13064">https://phabricator.kde.org/D13064</a></div></div><br /><div><strong>To: </strong>hpereiradacosta, ngraham, abetts<br /><strong>Cc: </strong>zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>