Review Request 127120: Morphingpopups: Add config to disable size and fade animations

David Rosca nowrep at gmail.com
Sun Feb 21 09:31:39 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127120/
-----------------------------------------------------------

(Updated Feb. 21, 2016, 9:31 a.m.)


Review request for Plasma.


Repository: kwin


Description (updated)
-------

Disabling both animations makes the effect look like the old Plasma tooltip animation.
Also don't animate when geometry didn't change + don't skip small steps as resize may happen in steps so we must animate all of them.


Diffs (updated)
-----

  effects/morphingpopups/package/contents/code/morphingpopups.js ce6ad86 
  effects/morphingpopups/package/contents/config/main.xml PRE-CREATION 
  effects/morphingpopups/package/contents/ui/config.ui PRE-CREATION 
  effects/morphingpopups/package/metadata.desktop f9b192c 

Diff: https://git.reviewboard.kde.org/r/127120/diff/


Testing
-------

Changing config from KCM needs KWin restart because even though `configChanged` is emitted, `readConfig` returns the old values (bug, or am I doing something wrong?).


Thanks,

David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160221/a4964507/attachment.html>


More information about the Plasma-devel mailing list