[Differential] [Commented On] D3408: Alt fix for D3407

[mart (Marco Martin)]

mart added a comment.


  in general +1, seems a bit cleaner one it's all well tested, it's ok for me

INLINE COMMENTS

> panelview.cpp:577
> +    else if (containment() &&  (containment()->status() == Plasma::Types::RequiresAttentionStatus )) {
> +//              || containment()->status() == Plasma::Types::HiddenStatus)) {
> +        autoHide = false;

why this got commented out? (that's the one that should be really checked thoroughly)

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D3408

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: mart, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161118/e0a3bd66/attachment.html>