--===============2581375818215906770== Content-Type: multipart/alternative; boundary="===============2051116477146313282==" --===============2051116477146313282== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable > On Jan. 21, 2012, 5:19 p.m., Philipp Knechtges wrote: > > Although I agree with you that the effects shouldn't be bothered with a= window that isn't shown once, > > I would prefer to make the emitted signals consistent. Your current app= roach has the shortcoming that > > it might not emit the windowAdded signal, but on the other hand it woul= d always emit windowClosed > > and windowDeleted. i hope not. those are connected in setupClientConnections()now being invoke= d by slotClientShown() which is not called unless the window is or got read= yForPainting() ie. the connection is cut off on the effectshandler - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103742/#review9984 ----------------------------------------------------------- On Jan. 20, 2012, 2:21 a.m., Thomas L=C3=BCbking wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103742/ > ----------------------------------------------------------- > = > (Updated Jan. 20, 2012, 2:21 a.m.) > = > = > Review request for kwin, Martin Gr=C3=A4=C3=9Flin and Philipp Knechtges. > = > = > Description > ------- > = > see summary. Makes animations smoother and would have prevented RR http:/= /git.reviewboard.kde.org/r/103733 ;-) > = > I'll keep it in my personal 4.8 branch and therefore won't care about whe= ther it's applied to 4.8.n or 4.9 :-P > = > = > Diffs > ----- > = > kwin/client.cpp 520ed85 = > kwin/composite.cpp 55a7144 = > kwin/effects.h f4ad2fb = > kwin/effects.cpp 6c252c9 = > kwin/events.cpp 26e0139 = > kwin/toplevel.h 693e6f3 = > kwin/toplevel.cpp ffe7f0c = > = > Diff: http://git.reviewboard.kde.org/r/103742/diff/diff > = > = > Testing > ------- > = > generic animations still work - better ;-) > = > = > Thanks, > = > Thomas L=C3=BCbking > = > --===============2051116477146313282== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/103742/ |
On January 21st, 2012, 5:19 p.m., Philipp K= nechtges wrote:
Although = I agree with you that the effects shouldn't be bothered with a window t= hat isn't shown once, I would prefer to make the emitted signals consistent. Your current approac= h has the shortcoming that it might not emit the windowAdded signal, but on the other hand it would al= ways emit windowClosed and windowDeleted.
i hope not.= those are connected in setupClientConnections()now being invoked by slotCl= ientShown() which is not called unless the window is or got readyForPaintin= g() ie. the connection is cut off on the effectshandler
- Thomas
On January 20th, 2012, 2:21 a.m., Thomas L=C3=BCbking wrote:
Review request for kwin, Martin Gr=C3=A4=C3=9Flin and Philipp Knechtge=
s.
By Thomas L=C3=BCbking.
Updated Jan. 20, 2012, 2:21 a.m. Descripti= on
Testing <= /h1>
Diffs=
|