> On Juli 3, 2015, 3:39 vorm., 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. :-)
> 
> Bart Schouten wrote:
>     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.
Our HIG say that CheckBoxes must be postive, ie. "enable" something not 
"disable" or "don't"


- Kai Uwe


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


On Juli 2, 2015, 9:23 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124234/
> -----------------------------------------------------------
> 
> (Updated Juli 2, 2015, 9:23 nachm.)
> 
> 
> 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