----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105283/#review14957 -----------------------------------------------------------
This review has been submitted with commit ac8552a15ec5a8881e4258113ec835c6c16e1a60 by David Edmundson to branch master. - Commit Hook 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
