> 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