> On June 18, 2012, 3:37 p.m., Viranch Mehta wrote: > > The button size and the hover appearance is different from the original > > one. The IconButton component was made to keep the look of the buttons > > consistent with the original version of the applet. Do we want to change > > this?
Valid argument for now, won't be valid when everything moves to QML/Plasma Components. You're maintainer, you have final say. If you want me to wait till 4.10 when more applets are QML based I will do. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105283/#review14839 ----------------------------------------------------------- On June 17, 2012, 7:52 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105283/ > ----------------------------------------------------------- > > (Updated June 17, 2012, 7:52 p.m.) > > > Review request for Plasma. > > > Description > ------- > > Current battery monitor implements it's own Button class, this previously > broke styles with theme text and overloads icon sizes and such. > > It's bad for applets to implement their own version of common classes as it > prevents consistency. > > (will fix the whitespace addition before commit) > > > Diffs > ----- > > plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml d4454c6 > plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml a2ab72a > > Diff: http://git.reviewboard.kde.org/r/105283/diff/ > > > Testing > ------- > > Checked applet looked ok. > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel