<table><tr><td style="">rooty 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/D17975">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17975#389580" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D17975#389580</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@filipf</a> wrote</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>I'd much rather if there was some padding added in Breeze and then you only fix the wrong spacing to right of the notification. Even if that wouldn't work, I don't get the sense we've fully explored other solutions yet.</p></div>
</blockquote>
<p>There are other problems though besides the right margin of the icon. The heading can't be lifted to stop the icon from towering over it because the theme provides its own padding and, in essence, <em>locks it into place</em>.</p>
<p>In other words, the padding here <strong>varies depending on the amount of notification content</strong>, which I don't think is something a theme can account for.</p>
<p>Modifying Breeze to add padding would restrict its ability to manipulate the notification content fully.</p>
<blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17975#389584" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D17975#389584</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>So what's going on here? Other themes add more padding? If so, I don't see how that's something we can possibly account for, and that shouldn't affect our ability to improve our own stuff.</p></div>
</blockquote>
<p>We may be able to account for it, but our hands are tied here I'm afraid because of the technical difficulties of making the Breeze theme doing what I want the notifications to do in this patch (vary padding by notification content).</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D17975#inline-99720">View Inline</a><span style="color: #4b4d51; font-weight: bold;">davidedmundson</span> wrote in <span style="color: #4b4d51; font-weight: bold;">NotificationItem.qml:33</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Much better.</p>
<p style="padding: 0; margin: 8px;">Only part I don't understand, then ship it! from me.</p>
<p style="padding: 0; margin: 8px;">Assuming there's no icon</p>
<p style="padding: 0; margin: 8px;">For multiline</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">1 small space above 1 small space under.</li>
</ul>
<p style="padding: 0; margin: 8px;">For one line we effectively get:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">0 margin on top, 0.5 under</li>
</ul>
<p style="padding: 0; margin: 8px;">Why the 0.5? Surely that would be inconsistent?</p>
<p style="padding: 0; margin: 8px;">(In all your examples the icon is bigger so we don't see it)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">You're right! I never tested it with just the heading, There's an extra 0.5 * units.smallSpacing of padding there. But I needed that 0.5 units to provide even padding to notifications that are "Heading + One line of description".</p>
<p style="padding: 0; margin: 8px;">Is there any way I can put in a condition that if it's just a heading that it not put that 0.5 * units.smallSpacing there?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17975">https://phabricator.kde.org/D17975</a></div></div><br /><div><strong>To: </strong>rooty, VDG, Plasma, ngraham<br /><strong>Cc: </strong>Codezela, abetts, filipf, davidedmundson, hein, ndavis, plasma-devel, kvanton, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>