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

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 the wrong

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

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

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 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

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

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: 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: 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

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 if

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