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

Kai Uwe Broulik <k...@privat.broulik.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/plas
                   |                            |ma-workspace/6c0b1e542c19e0
                   |                            |0e045074a01e4daf992ec59e56
   Version Fixed In|                            |5.7.0

--- Comment #4 from Kai Uwe Broulik <k...@privat.broulik.de> ---
Git commit 6c0b1e542c19e00e045074a01e4daf992ec59e56 by Kai Uwe Broulik.
Committed on 13/06/2016 at 08:10.
Pushed by broulik into branch 'master'.

[Device Notifier] Be more declarative

Instead of having a bunch of handlers in various places messing with various
settings,
try to use declarative bindings where possible.

* Remove separate passive timer, instead re-use the pendingDelegateRemoval
property
  which has its own timer. This way, when a device is unmounted, device
notifier stays
  active until the notification has cleared, however, when the device is yanked
out manually,
  it disappears right away
* Don't break binding on Plasmoid.active to ensure its state is always proper
depending on
  whether a device or not is present (the linked bug)
* Create tooltip declaratively rather than imperatively
FIXED-IN: 5.7.0

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

M  +0    -9   
applets/devicenotifier/package/contents/ui/FullRepresentation.qml
M  +22   -20   applets/devicenotifier/package/contents/ui/devicenotifier.qml

http://commits.kde.org/plasma-workspace/6c0b1e542c19e00e045074a01e4daf992ec59e56

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

Reply via email to