Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-21 Thread Johannes Tröscher
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105232/ --- (Updated June 20, 2012, 12:15 p.m.) Review request for Plasma. Changes -

Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-20 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105232/#review14920 --- This review has been submitted with commit a0f6888c23a650f7fe0

Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-20 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105232/#review14909 --- Ship it! can you push it? - Marco Martin On June 20, 2012,

Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-20 Thread Marco Martin
> On June 14, 2012, 6:18 p.m., Aaron J. Seigo wrote: > > plasma/declarativeimports/plasmacomponents/qml/private/DualStateButton.qml, > > lines 77-78 > > > > > > loos like this will work but i dislike how shadowLoade

Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-14 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105232/#review14727 --- plasma/declarativeimports/plasmacomponents/qml/private/DualSta

Re: Review Request: Plasm qml-components: also highlight the dualstate button on keyboard-focus

2012-06-13 Thread Johannes Tröscher
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105232/ --- (Updated June 12, 2012, 10:25 p.m.) Review request for Plasma. Summary (