https://bugs.kde.org/show_bug.cgi?id=370324

Eike Hein <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/plas
                   |                            |ma-desktop/a109124d0ac684a5
                   |                            |2c0167e28c21025b850cc6db
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from Eike Hein <[email protected]> ---
Git commit a109124d0ac684a52c0167e28c21025b850cc6db by Eike Hein.
Committed on 09/10/2016 at 09:45.
Pushed by hein into branch 'Plasma/5.8'.

Fix circular logic causing Pager to remain hidden

Summary:
The Pager applet would transition to HiddenStatus if PagerModel.count
was smaller than 2, however hiding the root item would set
PagerModel.enabled to false, dropping the count to 0 and stopping
the window system monitoring, thus never transitioning the applet
back to ActiveStatus if the desktop or activity count increased again.

This patch introduces a new PagerModel.shouldShowPager property which
is updated even when the model is disabled, then used to drive both the
applet status and the enabled state of the model.

Reviewers: #plasma, davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D2990

M  +1    -1    applets/pager/package/contents/ui/main.qml
M  +12   -0    applets/pager/plugin/pagermodel.cpp
M  +4    -0    applets/pager/plugin/pagermodel.h

http://commits.kde.org/plasma-desktop/a109124d0ac684a52c0167e28c21025b850cc6db

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to