Re: Review Request: Adding notify signals to FrameSvgItem

2012-08-21 Thread Christoph Feck
> On Aug. 12, 2012, 10:41 a.m., Christoph Feck wrote: > > As I understand it, the notify system is for attributes, where the object > > changes the value _itself_ (e.g. in response to user interaction), as > > opposed to where the application changed the value. > > > > In that light, an imageP

Re: Review Request: Make autohide panel do not intercept events in fullscreen app

2012-08-21 Thread Christoph Feck
> On Aug. 21, 2012, 10:24 a.m., Christoph Feck wrote: > > If this "feature" is going to be implemented, it should be configurable. If > > you are only using a mouse (or stylus on a Tablet PC), then you would have > > no way to reach the panel in full-screen mode. The better solution would be >

Re: Re: [RFC] Merging LightDM into KDE Workspaces

2012-08-21 Thread Alex Fiestas
Big +1 for me. Personally I have switched to lightDM long ago, the biggest reason why I pushed and I will push for it is because the codebase of the KDE greeter is small, compact and easy to hack on and the code of lightDM itself at least 8months back wasn't monstrous and I could read it easily

Re: Re: move plasma tools to plasmate

2012-08-21 Thread Alex Fiestas
I have changed my mind on this topic with the following conclusion (hope it is useful for future actions). The main reason why I don't like this movement is because it adds an extra dependency for the random KDE hacker that wants to do something. This hacker will compile everything so he will h

Re: Review Request: Pager QML port

2012-08-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106112/#review17812 --- found a small issue: when there is only one desktop, the pager

Re: Review Request: Pager QML port

2012-08-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106112/#review17811 --- Ship it! i'm using it since a while and all regressions seems

Re: Review Request: Add fade effect on the nowplaying text cycling

2012-08-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106095/#review17810 --- This review has been submitted with commit 48ccff55dabc7ad9a17

Re: Review Request: Add fade effect on the nowplaying text cycling

2012-08-21 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106095/#review17809 --- This review has been submitted with commit 30039bcfba7fe34bf41

Re: Dictionary KRunner: Complete

2012-08-21 Thread Jason A. Donenfeld
Buler? Buler? ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request: Add fade effect on the nowplaying text cycling

2012-08-21 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106095/#review17807 --- Ship it! Ship It! - Marco Martin On Aug. 20, 2012, 11:13 a.

Re: Review Request: Make autohide panel do not intercept events in fullscreen app

2012-08-21 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106110/#review17799 --- If this "feature" is going to be implemented, it should be conf

Re: Review Request: Make autohide panel do not intercept events in fullscreen app

2012-08-21 Thread Alexey Shmalko
> On Aug. 21, 2012, 4:22 a.m., Martin Gräßlin wrote: > > I do not see a reason why the hidden panel should not be allowed to go > > above a fullscreen window. True it's not possible for the normal panel > > case, but hidden panel is kind of special. > > > > Apart from that I think you modified

Re: Review Request: Make autohide panel do not intercept events in fullscreen app

2012-08-21 Thread Alexey Shmalko
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106110/ --- (Updated Aug. 21, 2012, 10:06 a.m.) Review request for Plasma. Changes -