<table><tr><td style="">ndavis 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/D19890">View Revision</a></tr></table><br /><div><div><p>Ok, so there is an overlap problem, but it's quite rare. It happens when an icon uses 100% of the available space in the bottom right corner (or left with RTL, I think).<br />
Here I changed the stop icon in KDevelop to the icon for Codelite: <a href="https://phabricator.kde.org/F7817068" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7817068: Screenshot_20191213_171219.png</a>.</p>

<p>I could get around this in a number of ways that all have their downsides:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Move the arrow to the very bottom right and make sure it's small enough that no clipping happens<ul class="remarkup-list">
<li class="remarkup-list-item">This looks bad and makes it harder to see the arrow</li>
</ul></li>
<li class="remarkup-list-item">Reduce the default icon size for flat buttons<ul class="remarkup-list">
<li class="remarkup-list-item">This is more consistent with non-flat buttons, but I like the bigger icons. They're more readable too.</li>
</ul></li>
<li class="remarkup-list-item">Increase the padding around buttons<ul class="remarkup-list">
<li class="remarkup-list-item">I like the current padding around toolbuttons</li>
</ul></li>
<li class="remarkup-list-item">Ignore the problem<ul class="remarkup-list">
<li class="remarkup-list-item">Will look fine with most icon themes, but bad in some cases such as the one described above.</li>
</ul></li>
</ul></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/D19890">https://phabricator.kde.org/D19890</a></div></div><br /><div><strong>To: </strong>hallas, VDG, Breeze, ngraham, abetts, ndavis<br /><strong>Cc: </strong>ndavis, abetts, hpereiradacosta, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, ahiemstra, mart<br /></div>