Review Request: Use common plasma components Tooltip in battery monitor

David Edmundson kde at davidedmundson.co.uk
Wed Jun 20 15:15:05 UTC 2012



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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120620/5c1fc5fb/attachment.html>


More information about the Plasma-devel mailing list