<table><tr><td style="">kossebau accepted this revision.<br />kossebau 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/D18936">View Revision</a></tr></table><br /><div><div><p>Well spotted. Seemed I had fooled myself as my test data always had both warnings & watches. And I never saw notices from real services when I tested, also not with the BBC service which I use only for real, so had not hit myself.</p>
<p>Can you push this to 5.12 branch and then merge all the way through 5.14 & 5.15 to master in the next 12 hours? 5.14 is interesting due to debian's upcoming release seeming to stick with 5.14, so such fixes would be good to also have in that branch, to ease anyone pulling improvements from upstream.<br />
If not, would push for you, so its also reaching the 5.12.8 release scheduled for Tuesday.</p>
<p>Works fine for what I quickly tested.</p>
<p>PS: With this working, also time to look again at using overlay emblems to the panel icon in case of warnings existing :)</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/D18936#inline-105366">View Inline</a><span style="color: #4b4d51; font-weight: bold;">NoticesView.qml:46</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; "> <span style="color: #aa4000">level:</span> <span style="color: #601200">4</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span style="color: #aa4000">Layout.alignment:</span> <span style="color: #004012">Qt</span><span class="p">.</span><span style="color: #004012">AlignHCenter</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Npt sure about this one, in my test data I always had >1 warnings/watches, and left aligned then somehow looked better.</p>
<p style="padding: 0; margin: 8px;">But fine to +1 this for now, we can always change it later if we finally get more real world usage by your fix and a better idea what looks good to most, other than the original developer :)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R114 Plasma Addons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18936">https://phabricator.kde.org/D18936</a></div></div><br /><div><strong>To: </strong>Zren, kossebau, Plasma<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>