Review Request: Use common plasma components Tooltip in battery monitor

[Viranch Mehta]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105283/#review14839
-----------------------------------------------------------


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?

- Viranch Mehta


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/20120618/c1842302/attachment.html>