Review Request 118573: Some tweaks in the logout screen

[Aleix Pol Gonzalez]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118573/#review59401
-----------------------------------------------------------


Ok, +1 from me, I'm not giving a shipit because I see graesslin has concerns about it skipping the WM.

Let's get this sorted.

- Aleix Pol Gonzalez


On June 6, 2014, 10:07 a.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118573/
> -----------------------------------------------------------
> 
> (Updated June 6, 2014, 10:07 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This does two things:
> * some tweaks in the logout theme, partly inverts theme colors and makes the logout/shutdown toolbuttons a group of 3 always visible exclusive-toggle buttons
> * add the flag bypasswindowmanagerhint to the main logout window: seems the only way here to obtain the proper size and position
> 
> (old screenshot to give the idea: http://wstaw.org/m/2014/06/05/plasma-desktopGg1731.png labels are actually of the proper color now)
> 
> 
> Diffs
> -----
> 
>   ksmserver/shutdowndlg.cpp 68efcf6 
>   lookandfeel/contents/components/BreezeBlock.qml 4093f5f 
>   lookandfeel/contents/components/BreezeHeading.qml PRE-CREATION 
>   lookandfeel/contents/components/BreezeLabel.qml PRE-CREATION 
>   lookandfeel/contents/components/LogoutOptions.qml PRE-CREATION 
>   lookandfeel/contents/components/LogoutScreen.qml 1f7825d 
> 
> Diff: https://git.reviewboard.kde.org/r/118573/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

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