Re: Review Request 109124: qml port currentappcontrol

2013-02-24 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109124/#review27964 --- Looks pretty good. Couple of pretty common mistakes to sort out

Re: Review Request 109124: qml port currentappcontrol

2013-02-24 Thread Greg T
> On Feb. 24, 2013, 12:27 p.m., Kai Uwe Broulik wrote: > > - the close/restore buttons don't glow on hover. I couldn't load a theme > > svg using the IconItem. Is that possible? > > Quoting from Marco Martin's blog: "You should not use it when your icon is > > actually meant to be a standalone

Re: Review Request 109124: qml port currentappcontrol

2013-02-24 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109124/#review27958 --- - the close/restore buttons don't glow on hover. I couldn't loa

Review Request 109124: qml port currentappcontrol

2013-02-24 Thread Greg T
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109124/ --- Review request for Plasma. Description --- Heya folks, This is a qml