Review Request 126776: [Panel Containment] Hide applet if its status is Hidden

Marco Martin notmart at gmail.com
Sun Jan 17 19:03:11 UTC 2016


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


Ship it!




Ship It!

- Marco Martin


On Jan. 17, 2016, 1:19 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126776/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2016, 1:19 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> With the newly introduced HiddenStatus we now have a way to hide panel applets in a controlled way instead of incorporating hacks like we currently do for the Pager.
> 
> When an applet sets its status to Hidden it will be removed from the layout and fully collapse as if it wasn't there. However, if you open the panel configuration, it shows up so you can still configure and permanently remove an applet, if so desired.
> 
> 
> Diffs
> -----
> 
>   containments/panel/contents/ui/main.qml bad6ba0 
> 
> Diff: https://git.reviewboard.kde.org/r/126776/diff/
> 
> 
> Testing
> -------
> 
> Pager completely collapses (Review 126777) when I only have one virtual desktop and re-appears if there are multiple. It also re-appears when I enter panel configuration.
> 
> This should also be used for the quick chat applet.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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


More information about the Plasma-devel mailing list