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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to