[prev in list] [next in list] [prev in thread] [next in thread]
List: kwin
Subject: Re: Review Request: JavaScript bindings for KWin Effects
From: Thomas_Lübking <thomas.luebking () web ! de>
Date: 2012-02-10 20:14:04
Message-ID: 20120210201404.15023.50762 () vidsolbach ! de
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103823/#review10494
-----------------------------------------------------------
Can one enforce a reference? ie. "int meta = setMeta(SourceAnchor, Top|Left, 1);" \
would be wrong but should be
int meta = 0;
setMeta(SourceAnchor, Top|Left, meta);
setMeta(TargetAnchor, Bottom|Right, meta);
> > animate(..., meta, ...)
Maybe (since the functionality will remain limited for scripting and "meta" is no \
longer a wildcard) adding functions as: static int meta(0);
resetMetaData(); // meta = 0
setSourceAnchor(int, a); // setMeta(SourceAnchor, a, meta);
setTargetAnchor(int); // .
setRotationAxis(Qt::Axis); // .
setSourceRelative(bool x, bool y); // .
setTargetRelative(bool x, bool y); // .
and scratch the meta parameter from the ::animate() function will be better for \
script usage (the wildcard meta is powerful, but operates on bit setting - what might \
be a bit out of scope here)
- Thomas Lübking
On Feb. 10, 2012, 8:46 a.m., Martin Gräßlin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103823/
> -----------------------------------------------------------
>
> (Updated Feb. 10, 2012, 8:46 a.m.)
>
>
> Review request for kwin.
>
>
> Description
> -------
>
> The bindings are based on AnimationEffect. So Thomas please have a look whether I \
> use it correctly :-) The method is exported multiple times as I did not want to \
> wrap the FPx2 class.
> An example script for fade effect:
> var duration = 250;
> effects.windowAdded.connect(function(w) {
> effect.animate(w, Effect.Opacity, duration, 1.0);
> });
> effects.windowClosed.connect(function(w) {
> effect.animate(w, Effect.Opacity, duration, 0.0, 1.0);
> });
>
> There is quite some work to be done. EffectWindow does not yet have properties, for \
> EffectsHandler probably some more methods might be exported.
> Unfortunately I had to change the signature of the signals as QtScript needs the \
> type being exported as a metatype and metatypes want full name.
>
> Diffs
> -----
>
> kwin/CMakeLists.txt c4b75c0
> kwin/effects.h 28ab72c
> kwin/effects.cpp 4f55799
> kwin/effects/CMakeLists.txt f09a855
> kwin/effects/blur/blur.h 65ba381
> kwin/effects/blur/blur.cpp 06c6854
> kwin/effects/boxswitch/boxswitch.h cfb6b47
> kwin/effects/boxswitch/boxswitch.cpp fac6b3f
> kwin/effects/coverswitch/coverswitch.h b49a36e
> kwin/effects/coverswitch/coverswitch.cpp bdf2fbe
> kwin/effects/cube/cubeslide.h ffbda88
> kwin/effects/cube/cubeslide.cpp ed1f518
> kwin/effects/dashboard/dashboard.h 46d3e98
> kwin/effects/dashboard/dashboard.cpp 4f6dc65
> kwin/effects/desktopgrid/desktopgrid.h b420389
> kwin/effects/desktopgrid/desktopgrid.cpp 925d2f1
> kwin/effects/dialogparent/dialogparent.h 439c67e
> kwin/effects/dialogparent/dialogparent.cpp ac9493a
> kwin/effects/diminactive/diminactive.h 24a80de
> kwin/effects/diminactive/diminactive.cpp 6978f36
> kwin/effects/dimscreen/dimscreen.h ee8c114
> kwin/effects/dimscreen/dimscreen.cpp fd4c15a
> kwin/effects/explosion/explosion.h 8a343c7
> kwin/effects/explosion/explosion.cpp dc9cd14
> kwin/effects/fade/CMakeLists.txt 865afb7
> kwin/effects/fade/fade.cpp 213516b
> kwin/effects/fade/fade.desktop 3b29244
> kwin/effects/fade/fade.h 38c0040
> kwin/effects/fade/package/contents/code/main.js PRE-CREATION
> kwin/effects/fade/package/contents/config/main.xml PRE-CREATION
> kwin/effects/fade/package/metadata.desktop PRE-CREATION
> kwin/effects/fadedesktop/CMakeLists.txt 61bd4fc
> kwin/effects/fadedesktop/fadedesktop.h d5d8b36
> kwin/effects/fadedesktop/fadedesktop.cpp 5d71ef7
> kwin/effects/fadedesktop/fadedesktop.desktop 8b64b70
> kwin/effects/fadedesktop/package/contents/code/main.js PRE-CREATION
> kwin/effects/fadedesktop/package/metadata.desktop PRE-CREATION
> kwin/effects/fallapart/fallapart.h acc1446
> kwin/effects/fallapart/fallapart.cpp 5db5ee7
> kwin/effects/flipswitch/flipswitch.h 559e88b
> kwin/effects/flipswitch/flipswitch.cpp f7f4307
> kwin/effects/glide/glide.h 0003345
> kwin/effects/glide/glide.cpp 99a8c8b
> kwin/effects/highlightwindow/highlightwindow.h 8b46bd4
> kwin/effects/highlightwindow/highlightwindow.cpp f048cfb
> kwin/effects/invert/invert.h 3dc49fb
> kwin/effects/invert/invert.cpp 0ef1545
> kwin/effects/login/login.h db30044
> kwin/effects/login/login.cpp a9da287
> kwin/effects/logout/logout.h f08152f
> kwin/effects/logout/logout.cpp 4e97ebe
> kwin/effects/magiclamp/magiclamp.h 8000081
> kwin/effects/magiclamp/magiclamp.cpp f7f73b4
> kwin/effects/minimizeanimation/minimizeanimation.h 27e0dd3
> kwin/effects/minimizeanimation/minimizeanimation.cpp 44d15ec
> kwin/effects/presentwindows/presentwindows.h 08aa262
> kwin/effects/presentwindows/presentwindows.cpp dc4d5c5
> kwin/effects/resize/resize.h 0411d6d
> kwin/effects/resize/resize.cpp 84bdd7f
> kwin/effects/scalein/scalein.h 8ae7504
> kwin/effects/scalein/scalein.cpp 25d78df
> kwin/effects/screenshot/screenshot.h 22385a3
> kwin/effects/screenshot/screenshot.cpp 6bd18e3
> kwin/effects/sheet/sheet.h 7ec739b
> kwin/effects/sheet/sheet.cpp 9156f0d
> kwin/effects/slideback/slideback.h 0cba556
> kwin/effects/slideback/slideback.cpp 380b3d5
> kwin/effects/slidingpopups/slidingpopups.h eca9e2e
> kwin/effects/slidingpopups/slidingpopups.cpp 8fb0608
> kwin/effects/snaphelper/snaphelper.h ee60f8f
> kwin/effects/snaphelper/snaphelper.cpp a30cc60
> kwin/effects/taskbarthumbnail/taskbarthumbnail.h 77e5294
> kwin/effects/taskbarthumbnail/taskbarthumbnail.cpp 1ce6233
> kwin/effects/thumbnailaside/thumbnailaside.h 323e2fd
> kwin/effects/thumbnailaside/thumbnailaside.cpp 3a1b4e0
> kwin/effects/translucency/translucency.h fa7b85c
> kwin/effects/translucency/translucency.cpp 4e6d58d
> kwin/effects/windowgeometry/windowgeometry.h ba20c05
> kwin/effects/windowgeometry/windowgeometry.cpp a9fd796
> kwin/effects/wobblywindows/wobblywindows.h d4844d3
> kwin/effects/wobblywindows/wobblywindows.cpp c7d30fe
> kwin/libkwineffects/kwinanimationeffect.h 106155c
> kwin/libkwineffects/kwinanimationeffect.cpp 00a7887
> kwin/libkwineffects/kwineffects.h 491ec58
> kwin/libkwineffects/kwineffects.cpp 3f29062
> kwin/scripting/scriptedeffect.h PRE-CREATION
> kwin/scripting/scriptedeffect.cpp PRE-CREATION
> kwin/thumbnailitem.cpp 73441a0
>
> Diff: http://git.reviewboard.kde.org/r/103823/diff/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Martin Gräßlin
>
>
[Attachment #5 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/103823/">http://git.reviewboard.kde.org/r/103823/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Can one enforce a \
reference? ie. "int meta = setMeta(SourceAnchor, Top|Left, 1);" would be \
wrong but should be
int meta = 0;
setMeta(SourceAnchor, Top|Left, meta);
setMeta(TargetAnchor, Bottom|Right, meta);
> > animate(..., meta, ...)
Maybe (since the functionality will remain limited for scripting and "meta" \
is no longer a wildcard) adding functions as: static int meta(0);
resetMetaData(); // meta = 0
setSourceAnchor(int, a); // setMeta(SourceAnchor, a, meta);
setTargetAnchor(int); // .
setRotationAxis(Qt::Axis); // .
setSourceRelative(bool x, bool y); // .
setTargetRelative(bool x, bool y); // .
and scratch the meta parameter from the ::animate() function will be better for \
script usage (the wildcard meta is powerful, but operates on bit setting - what might \
be a bit out of scope here)</pre> <br />
<p>- Thomas</p>
<br />
<p>On February 10th, 2012, 8:46 a.m., Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for kwin.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated Feb. 10, 2012, 8:46 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">The bindings are based on AnimationEffect. So Thomas please have a look \
whether I use it correctly :-) The method is exported multiple times as I did not \
want to wrap the FPx2 class.
An example script for fade effect:
var duration = 250;
effects.windowAdded.connect(function(w) {
effect.animate(w, Effect.Opacity, duration, 1.0);
});
effects.windowClosed.connect(function(w) {
effect.animate(w, Effect.Opacity, duration, 0.0, 1.0);
});
There is quite some work to be done. EffectWindow does not yet have properties, for \
EffectsHandler probably some more methods might be exported.
Unfortunately I had to change the signature of the signals as QtScript needs the type \
being exported as a metatype and metatypes want full name.</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kwin/CMakeLists.txt <span style="color: grey">(c4b75c0)</span></li>
<li>kwin/effects.h <span style="color: grey">(28ab72c)</span></li>
<li>kwin/effects.cpp <span style="color: grey">(4f55799)</span></li>
<li>kwin/effects/CMakeLists.txt <span style="color: grey">(f09a855)</span></li>
<li>kwin/effects/blur/blur.h <span style="color: grey">(65ba381)</span></li>
<li>kwin/effects/blur/blur.cpp <span style="color: grey">(06c6854)</span></li>
<li>kwin/effects/boxswitch/boxswitch.h <span style="color: \
grey">(cfb6b47)</span></li>
<li>kwin/effects/boxswitch/boxswitch.cpp <span style="color: \
grey">(fac6b3f)</span></li>
<li>kwin/effects/coverswitch/coverswitch.h <span style="color: \
grey">(b49a36e)</span></li>
<li>kwin/effects/coverswitch/coverswitch.cpp <span style="color: \
grey">(bdf2fbe)</span></li>
<li>kwin/effects/cube/cubeslide.h <span style="color: grey">(ffbda88)</span></li>
<li>kwin/effects/cube/cubeslide.cpp <span style="color: grey">(ed1f518)</span></li>
<li>kwin/effects/dashboard/dashboard.h <span style="color: \
grey">(46d3e98)</span></li>
<li>kwin/effects/dashboard/dashboard.cpp <span style="color: \
grey">(4f6dc65)</span></li>
<li>kwin/effects/desktopgrid/desktopgrid.h <span style="color: \
grey">(b420389)</span></li>
<li>kwin/effects/desktopgrid/desktopgrid.cpp <span style="color: \
grey">(925d2f1)</span></li>
<li>kwin/effects/dialogparent/dialogparent.h <span style="color: \
grey">(439c67e)</span></li>
<li>kwin/effects/dialogparent/dialogparent.cpp <span style="color: \
grey">(ac9493a)</span></li>
<li>kwin/effects/diminactive/diminactive.h <span style="color: \
grey">(24a80de)</span></li>
<li>kwin/effects/diminactive/diminactive.cpp <span style="color: \
grey">(6978f36)</span></li>
<li>kwin/effects/dimscreen/dimscreen.h <span style="color: \
grey">(ee8c114)</span></li>
<li>kwin/effects/dimscreen/dimscreen.cpp <span style="color: \
grey">(fd4c15a)</span></li>
<li>kwin/effects/explosion/explosion.h <span style="color: \
grey">(8a343c7)</span></li>
<li>kwin/effects/explosion/explosion.cpp <span style="color: \
grey">(dc9cd14)</span></li>
<li>kwin/effects/fade/CMakeLists.txt <span style="color: grey">(865afb7)</span></li>
<li>kwin/effects/fade/fade.cpp <span style="color: grey">(213516b)</span></li>
<li>kwin/effects/fade/fade.desktop <span style="color: grey">(3b29244)</span></li>
<li>kwin/effects/fade/fade.h <span style="color: grey">(38c0040)</span></li>
<li>kwin/effects/fade/package/contents/code/main.js <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/effects/fade/package/contents/config/main.xml <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/effects/fade/package/metadata.desktop <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/effects/fadedesktop/CMakeLists.txt <span style="color: \
grey">(61bd4fc)</span></li>
<li>kwin/effects/fadedesktop/fadedesktop.h <span style="color: \
grey">(d5d8b36)</span></li>
<li>kwin/effects/fadedesktop/fadedesktop.cpp <span style="color: \
grey">(5d71ef7)</span></li>
<li>kwin/effects/fadedesktop/fadedesktop.desktop <span style="color: \
grey">(8b64b70)</span></li>
<li>kwin/effects/fadedesktop/package/contents/code/main.js <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/effects/fadedesktop/package/metadata.desktop <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/effects/fallapart/fallapart.h <span style="color: \
grey">(acc1446)</span></li>
<li>kwin/effects/fallapart/fallapart.cpp <span style="color: \
grey">(5db5ee7)</span></li>
<li>kwin/effects/flipswitch/flipswitch.h <span style="color: \
grey">(559e88b)</span></li>
<li>kwin/effects/flipswitch/flipswitch.cpp <span style="color: \
grey">(f7f4307)</span></li>
<li>kwin/effects/glide/glide.h <span style="color: grey">(0003345)</span></li>
<li>kwin/effects/glide/glide.cpp <span style="color: grey">(99a8c8b)</span></li>
<li>kwin/effects/highlightwindow/highlightwindow.h <span style="color: \
grey">(8b46bd4)</span></li>
<li>kwin/effects/highlightwindow/highlightwindow.cpp <span style="color: \
grey">(f048cfb)</span></li>
<li>kwin/effects/invert/invert.h <span style="color: grey">(3dc49fb)</span></li>
<li>kwin/effects/invert/invert.cpp <span style="color: grey">(0ef1545)</span></li>
<li>kwin/effects/login/login.h <span style="color: grey">(db30044)</span></li>
<li>kwin/effects/login/login.cpp <span style="color: grey">(a9da287)</span></li>
<li>kwin/effects/logout/logout.h <span style="color: grey">(f08152f)</span></li>
<li>kwin/effects/logout/logout.cpp <span style="color: grey">(4e97ebe)</span></li>
<li>kwin/effects/magiclamp/magiclamp.h <span style="color: \
grey">(8000081)</span></li>
<li>kwin/effects/magiclamp/magiclamp.cpp <span style="color: \
grey">(f7f73b4)</span></li>
<li>kwin/effects/minimizeanimation/minimizeanimation.h <span style="color: \
grey">(27e0dd3)</span></li>
<li>kwin/effects/minimizeanimation/minimizeanimation.cpp <span style="color: \
grey">(44d15ec)</span></li>
<li>kwin/effects/presentwindows/presentwindows.h <span style="color: \
grey">(08aa262)</span></li>
<li>kwin/effects/presentwindows/presentwindows.cpp <span style="color: \
grey">(dc4d5c5)</span></li>
<li>kwin/effects/resize/resize.h <span style="color: grey">(0411d6d)</span></li>
<li>kwin/effects/resize/resize.cpp <span style="color: grey">(84bdd7f)</span></li>
<li>kwin/effects/scalein/scalein.h <span style="color: grey">(8ae7504)</span></li>
<li>kwin/effects/scalein/scalein.cpp <span style="color: grey">(25d78df)</span></li>
<li>kwin/effects/screenshot/screenshot.h <span style="color: \
grey">(22385a3)</span></li>
<li>kwin/effects/screenshot/screenshot.cpp <span style="color: \
grey">(6bd18e3)</span></li>
<li>kwin/effects/sheet/sheet.h <span style="color: grey">(7ec739b)</span></li>
<li>kwin/effects/sheet/sheet.cpp <span style="color: grey">(9156f0d)</span></li>
<li>kwin/effects/slideback/slideback.h <span style="color: \
grey">(0cba556)</span></li>
<li>kwin/effects/slideback/slideback.cpp <span style="color: \
grey">(380b3d5)</span></li>
<li>kwin/effects/slidingpopups/slidingpopups.h <span style="color: \
grey">(eca9e2e)</span></li>
<li>kwin/effects/slidingpopups/slidingpopups.cpp <span style="color: \
grey">(8fb0608)</span></li>
<li>kwin/effects/snaphelper/snaphelper.h <span style="color: \
grey">(ee60f8f)</span></li>
<li>kwin/effects/snaphelper/snaphelper.cpp <span style="color: \
grey">(a30cc60)</span></li>
<li>kwin/effects/taskbarthumbnail/taskbarthumbnail.h <span style="color: \
grey">(77e5294)</span></li>
<li>kwin/effects/taskbarthumbnail/taskbarthumbnail.cpp <span style="color: \
grey">(1ce6233)</span></li>
<li>kwin/effects/thumbnailaside/thumbnailaside.h <span style="color: \
grey">(323e2fd)</span></li>
<li>kwin/effects/thumbnailaside/thumbnailaside.cpp <span style="color: \
grey">(3a1b4e0)</span></li>
<li>kwin/effects/translucency/translucency.h <span style="color: \
grey">(fa7b85c)</span></li>
<li>kwin/effects/translucency/translucency.cpp <span style="color: \
grey">(4e6d58d)</span></li>
<li>kwin/effects/windowgeometry/windowgeometry.h <span style="color: \
grey">(ba20c05)</span></li>
<li>kwin/effects/windowgeometry/windowgeometry.cpp <span style="color: \
grey">(a9fd796)</span></li>
<li>kwin/effects/wobblywindows/wobblywindows.h <span style="color: \
grey">(d4844d3)</span></li>
<li>kwin/effects/wobblywindows/wobblywindows.cpp <span style="color: \
grey">(c7d30fe)</span></li>
<li>kwin/libkwineffects/kwinanimationeffect.h <span style="color: \
grey">(106155c)</span></li>
<li>kwin/libkwineffects/kwinanimationeffect.cpp <span style="color: \
grey">(00a7887)</span></li>
<li>kwin/libkwineffects/kwineffects.h <span style="color: \
grey">(491ec58)</span></li>
<li>kwin/libkwineffects/kwineffects.cpp <span style="color: \
grey">(3f29062)</span></li>
<li>kwin/scripting/scriptedeffect.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/scripting/scriptedeffect.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>kwin/thumbnailitem.cpp <span style="color: grey">(73441a0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103823/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
kwin mailing list
kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic