<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D27979">View Revision</a></tr></table><br /><div><div><p>The padding looks much better with default font settings: <a href="https://phabricator.kde.org/F8211692" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8211692: Screenshot_20200402_090740.png</a></p>

<p>I'm using 11pt Ubuntu though, not 10pt Noto Sans. So now I'm thinking that there's some issue with making the padding dependent on the font. Still, that's clearly unrelated to this patch (it just exposes a pre-existing issue) so I say go for it! Bonus points for investigating the font padding issue in a follow-up patch. :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>notification_toparea</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27979">https://phabricator.kde.org/D27979</a></div></div><br /><div><strong>To: </strong>niccolove, broulik, ngraham<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>