<table><tr><td style="">loh.tar 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/D17556">View Revision</a></tr></table><br /><div><div><p>Just my 2Cents</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I think the zero left margin looks odd, it should at least align with the icons.</p></blockquote>



<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>i think it should either be aligned to the left of the icons, or to the left of the text items</p></blockquote>

<p>I agree to both of you, but the title "Make Status and Notification popup consistent with others" is right. As it is now it fit perfect with the others when you click through your list of status bar icons. So with your suggestion should the others modified too.</p>

<p>The only thing I would like to see is that the applet is not hidden when you click on it while some other was shown (1). But that's out of the scope from this patch.</p>

<p>(1) To clarify, assume always nothing is shown:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Click Notification (Shown), Click again (Hidden) -> OK</li>
<li class="remarkup-list-item">Click Notification (Shown), Click e.g. Volume (Glides to Volume), Click Notification (Hidden) -> Bad! Should be shown and hidden when clicked again</li>
</ul></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/D17556">https://phabricator.kde.org/D17556</a></div></div><br /><div><strong>To: </strong>shubham, VDG, broulik, Plasma, mart, hein<br /><strong>Cc: </strong>loh.tar, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>