Review Request: use Plasma::Dialog for brightness osd

[Kai Uwe Broulik]

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


Just applied the patch from Review 107983 and your patch resolves the issue. Would you mind if I use that code to fix KMix OSD?


powerdevil/daemon/brightnessosdwidget.cpp
<http://git.reviewboard.kde.org/r/108222/#comment19062>

    Originally the meterWidth was iconSize * 12 resulting in the total width of (roughly):
    
    iconSize * 13 + m_volumeLabel->nativeWidget()->size().width();


- Kai Uwe Broulik


On Jan. 6, 2013, 1:36 p.m., Xuetian Weng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108222/
> -----------------------------------------------------------
> 
> (Updated Jan. 6, 2013, 1:36 p.m.)
> 
> 
> Review request for kde-workspace, Plasma and Aaron J. Seigo.
> 
> 
> Description
> -------
> 
> well, you know.. this would fix the shadow problem, and clean up code actually.
> 
> 
> Diffs
> -----
> 
>   powerdevil/daemon/brightnessosdwidget.h ef08903 
>   powerdevil/daemon/brightnessosdwidget.cpp e4cf80a 
> 
> Diff: http://git.reviewboard.kde.org/r/108222/diff/
> 
> 
> Testing
> -------
> 
> locally tested, no problem.
> 
> 
> Thanks,
> 
> Xuetian Weng
> 
>

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