> On Oct. 25, 2015, 10:05 p.m., Thomas Pfeiffer wrote: > > As always: If you want design or usability input, please provide a > > screenshot. We cannot read the code. Thanks. > > Eike Hein wrote: > Thomas, all this does is not grey out the text in Task Manager button > labels when a window is minimized. See also the long discussion in the > referenced bug ticket. > > Thomas Pfeiffer wrote: > Thanks, I just wanted to make sure what it specifically does. > In that case, I am against the patch as it is. There should be some way > of distinguishing minimized from non-minimized tasks. > I agree with the problem, but I do not agree with the solution. > > Would it be possible to instead of hardcoded greying-out (which I also > agree is not the right representation, because greying out is reserved for > disabled controls) allow the Plasma theme to define the visual representation? > In the Breeze theme, it would make sense (as suggested in the bug thread) > to use a different color for the highlight bar instead of changing the > representation of the task. Other themes might choose differen > representations, or just don't distinguish them at all.
Other solutions: size (e.g. make the button smaller), text (brackets for instance), position (special area outside the normal task bar), subtile grey out (icon only), or add some decorator to the icon. The color might work well - and would probably be the easiest way, but I'm afraid of the desire to show more states like maximized, docked, active, different virtual screen etc. with the need to remember all the colors. Personally I'd like the position very much since minimizing a window means to not include it into the workflow, more or less. It's like the most missing feature of minimizing into tray. However that would be quite a big change. - Heiko ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124675/#review87384 ----------------------------------------------------------- On Oct. 23, 2015, 11:16 a.m., Gregor Mi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124675/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2015, 11:16 a.m.) > > > Review request for Plasma, KDE Usability and Jens Reuterberg. > > > Bugs: 311991 > https://bugs.kde.org/show_bug.cgi?id=311991 > > > Repository: plasma-desktop > > > Description > ------- > > This fixes Bug 311991. Though I am not sure if there are side effects which I > am not aware of. > > > Diffs > ----- > > applets/taskmanager/package/contents/ui/Task.qml > f655801ab1f7b1a9a915f35149c858f0ede22a29 > > Diff: https://git.reviewboard.kde.org/r/124675/diff/ > > > Testing > ------- > > > Thanks, > > Gregor Mi > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
