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

kwin/scripts/videowall/contents/code/main.js (Diff revision 1)
		function isVideoPlayer(client) {
		10	}

if (client.resourceName == xv) { //mplayer
   return true;
}

if (client.resourceName == ffplay) {
   return true;
}

should iirc also hit some mediacenter stuff

kwin/scripts/videowall/contents/code/main.js (Diff revision 1)
		function isVideoPlayer(client) {
		14	workspace.clientFullScreenSet.connect(function(client, set) {

concerning readability - i had to look twice to understand that this doesn't lack a closing parenthesis but is an inline function definition ;-P

- Thomas

On February 18th, 2012, 11:46 a.m., Martin Gräßlin wrote:

Review request for kwin.

By Martin Gräßlin.

Updated Feb. 18, 2012, 11:46 a.m.

Description

General: I changed KWin scripts to use Plasma Packages and install services. This allows us to ship scripts pre-installed but not enabled. This is a first such script.

This script allows to have video fullscreen windows span all attached screens creating a video wall.
    
This replaces the removed functionality that you could disable xinerama support inside KWin for fullscreen windows.

Currently vlc, smplayer and dragon player are recognized as video players.

Diffs

kwin/CMakeLists.txt (74e9879)
kwin/scripts/CMakeLists.txt (PRE-CREATION)
kwin/scripts/videowall/contents/code/main.js (PRE-CREATION)
kwin/scripts/videowall/metadata.desktop (PRE-CREATION)

View Diff