<table><tr><td style="">hpereiradacosta created this revision.<br />hpereiradacosta added reviewers: Breeze, ndavis.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />hpereiradacosta requested review of this revision.
</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/D23296">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Following on <a href="https://phabricator.kde.org/D23169" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D23169</a>, it turns out one can simplify the toolbuttons with menu rendering strongly by rendering the whole frame at once, and removing the code for detecting the presence of the menu alltogether. This makes for cleaner code with no visual difference with respect to how it should look (and no 'double-shadow').</p></div></div><br /><div><strong>TEST PLAN</strong><div><p><a href="https://phabricator.kde.org/F7264651" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7264651: Screenshot_20190820_174107.png</a><br />
This is how toolbuttons with menu appear with this patch. (no change with respect to without)</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/D23296">https://phabricator.kde.org/D23296</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kstyle/breezestyle.cpp</div></div></div><br /><div><strong>To: </strong>hpereiradacosta, Breeze, ndavis<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>