> On jul 3, 2015, 3:39 a.m., Andrew Lake wrote:
> > Yeah, let's not bother with the dimming. It just introduces another 
> > visualization for a state that's ostensibly disabled. Hide or no hide, 
> > there is no try. :-)

Hi, xen here. Hope this works. Doing everything on a smartphone, kinda slow.

I was writing that the new layout looks better indeed. I would still suggest 
putting the text+checkbox below the list, but make sure it stays on screen (ie. 
scroll the list not the entire pane). By putting it below it kinda summarizes 
the list and gets the aattention it deserves. Also, the wording. I'd suggest 
wording it negatively. Ie. not "Always show all icons" but "Don't hide unused 
icons". This makes it congruent with the user's perspective, which is that 
generally stuff is hidden (not shown). Give it a try, it will read much easier 
and be easier comprehensible. To reframe "hiding icons" is the familiar 
proposition, "showing" isn't. (Show what? What icons? Icons are already shown 
aren't they? Oh - the /hidden/ icons!) So that's what the user is after if she 
comes here.


- Bart


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


On jul 2, 2015, 9:23 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124234/
> -----------------------------------------------------------
> 
> (Updated jul 2, 2015, 9:23 p.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Bugs: 349812
>     https://bugs.kde.org/show_bug.cgi?id=349812
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This adds an option to always show all system tray icons and never collapse 
> them into the popup. I've seen a couple of people already requesting this.
> 
> 
> Diffs
> -----
> 
>   applets/systemtray/package/contents/config/main.xml 8e8996c 
>   applets/systemtray/package/contents/ui/ConfigEntries.qml 8b6ec28 
>   applets/systemtray/package/contents/ui/TaskDelegate.qml 221c6e7 
>   applets/systemtray/package/contents/ui/main.qml e794d1c 
>   applets/systemtray/plugin/host.h bd9728b 
>   applets/systemtray/plugin/host.cpp 5c897fa 
> 
> Diff: https://git.reviewboard.kde.org/r/124234/diff/
> 
> 
> Testing
> -------
> 
> Enabled and disabled the option and tray icons along with the arrow came and 
> went, started music player and media controller properly appeared and 
> disappeared, started and quit KMail and its icon properla appeared and 
> disappeared with both options.
> 
> 
> File Attachments
> ----------------
> 
> Settings dialog (disabled)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/50ae2a2e-8052-4d87-85c3-5ad8aa1da1f4__trayalways_dialog.png
> Settings dialog (enabled)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/d79b67e9-9950-4fb7-97a9-b89b93caeeb7__trayalways_dialogenabled.png
> System Tray (disabled)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/1b122131-88b0-410c-8532-8dbb74cad5db__trayalways_disabled.png
> System Tray (enabled)
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/3668a352-65c1-4d34-94d8-bc7141ce784e__trayalways_enabled.png
> Better dialog layout
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/cf2a921d-a4e3-4e94-b19d-3c6b0d3f3335__trayalways_dialogbetter.png
> Dim System Tray
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2015/07/02/57d2c759-97f8-4c44-9f00-c08558de9a9d__trayalways_passive.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

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

Reply via email to