-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5529/
-----------------------------------------------------------

Review request for kwin, Plasma and Lubos Lunak.


Summary
-------

Implemented an OpenGL based Startup Feedback effect to replace the 
functionality provided by StartupId in KRunner when effects are available. The 
effect supports the three modes bouncing, blinking and passive in exactly the 
same way as the current existing functionality.

Using an effect has several advantages:
 * we do not need to remove the alpha channel from the icons
 * no need to move windows around, shape and raise them
 * no need to prepare several pixmaps for blinking mode as OpenGL provides the 
same functionality using texture modulation (have I ever mentioned that I hate 
texture modulation?)

The effect is completely controlled by KRunner through DBus, when a startup is 
added the effect get's activated, when the last current startup is removed the 
effect gets deactivated again. The DBus call has to pass the type (bouncing, 
blinking or passive) and the icon name to the effect.


Diffs
-----

  trunk/KDE/kdebase/workspace/krunner/startupid.h 1182634 
  trunk/KDE/kdebase/workspace/krunner/startupid.cpp 1182634 
  trunk/KDE/kdebase/workspace/kwin/effects/CMakeLists.txt 1182656 
  trunk/KDE/kdebase/workspace/kwin/effects/startupfeedback/CMakeLists.txt 
PRE-CREATION 
  trunk/KDE/kdebase/workspace/kwin/effects/startupfeedback/startupfeedback.h 
PRE-CREATION 
  trunk/KDE/kdebase/workspace/kwin/effects/startupfeedback/startupfeedback.cpp 
PRE-CREATION 
  
trunk/KDE/kdebase/workspace/kwin/effects/startupfeedback/startupfeedback.desktop
 PRE-CREATION 

Diff: http://svn.reviewboard.kde.org/r/5529/diff


Testing
-------

* StartupId uses KWin effect when effect is present
* Changing mode works
* StartupId uses "legacy" window when effect is not present.


Thanks,

Martin

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to