<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124675/">https://git.reviewboard.kde.org/r/124675/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 25th, 2015, 10:05 p.m. UTC, <b>Thomas Pfeiffer</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">As always: If you want design or usability input, please provide a screenshot. We cannot read the code. Thanks.</p></pre>
</blockquote>
<p>On October 26th, 2015, 10:33 a.m. UTC, <b>Eike Hein</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">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.</p></pre>
</blockquote>
<p>On October 26th, 2015, 7:52 p.m. UTC, <b>Thomas Pfeiffer</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">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.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">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.</p></pre>
</blockquote>
<p>On October 26th, 2015, 8:10 p.m. UTC, <b>Heiko Tietze</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">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.</p></pre>
</blockquote>
<p>On October 26th, 2015, 9:17 p.m. UTC, <b>Eike Hein</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Changing size or position is jarring because it causes a lot of movement on the bar. Desaturating the icon is unpopular with users too, since the color information is desired to recognize an icon quickly.</p></pre>
</blockquote>
<p>On October 26th, 2015, 9:19 p.m. UTC, <b>Christoph Feck</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If I understand the bug report correctly, the issue was with the icon only, so the text graying could be kept, and the icon graying removed.</pre>
</blockquote>
<p>On October 26th, 2015, 9:39 p.m. UTC, <b>Gregor Mi</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">See e.g. comment 33 "Faded text and colours makes this difficult". So it is also about the text.</p></pre>
</blockquote>
<p>On October 27th, 2015, 2:33 a.m. UTC, <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fwiw, I, as an ordinary user, do not see any difference between the window being minimalized and the window not being on top (active window). My laptop screen is small~ish (13") and I always work in maximized windows. Therefore, representing the minimized state in the taskbar in any way bears absolutely no value for me, because if I want to swtich to any other window, it makes no difference if it just moves in the stack of visible windows or if the window is unminimized. The goal is to get the window to the top/make it an active window, I do not care where it comes from. Or, more importantly, I do not need to care.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Have we considered a usecase like that? I kind of question the whole usefulness of representing the minimized state, switching to it makes no difference besides the unminimizing animation, it just makes it harder to find. I mean, I've never looked at the taskbar searching Konsole and thought to myself "ah right, Konsole is now minimized, so now what?". What would our personas do? Why is it so important to be able to tell if the window is minimized or not?</p></pre>
</blockquote>
<p>On October 27th, 2015, 3:51 a.m. UTC, <b>Eli MacKenzie</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">My own use-case for a visible minimized state is as a flag. As far as I know there is no means to associate arbitrary data to a taskbar entry, so I'll minimize a window when it is "flagged". For example, if a browser window stays minimized "too long" I'll know I have to go bookmark some of its tabs and close it. I tend to remain logged in for months at a time.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">FWIW I have a 24" 16:10 monitor and use maximized windows.</p></pre>
</blockquote>
<p>On October 27th, 2015, 6:52 p.m. UTC, <b>Gregor Mi</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks for the input. If I may summarize: we have the ordinary user view vs. the power user view:
- For the ordinary (or impaired) user a long-outstanding issue would be fixed.
- The power user would like to keep a feature to be enabled by default.
How will this be moderated?</p></pre>
</blockquote>
<p>On October 27th, 2015, 7:27 p.m. UTC, <b>Heiko Tietze</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Simplest solution is to accept your patch (aka opt-in the minimized state). Ship it.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Heiko's perception that we now have an "opt-in" solution was based on a misunderstanding.
Nevertheless, we agreed that since greying out is definitely not the correct visualization and your patch fixes this, you can ship it.
If the VDG comes up with a better solution, we'll propose it. Until then, having minimized windows look the same as non-minimized ones is definitely better than what we currently have.</p></pre>
<br />
<p>- Thomas</p>
<br />
<p>On October 23rd, 2015, 11:16 a.m. UTC, Gregor Mi wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma, KDE Usability and Jens Reuterberg.</div>
<div>By Gregor Mi.</div>
<p style="color: grey;"><i>Updated Oct. 23, 2015, 11:16 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=311991">311991</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This fixes Bug 311991. Though I am not sure if there are side effects which I am not aware of.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>applets/taskmanager/package/contents/ui/Task.qml <span style="color: grey">(f655801ab1f7b1a9a915f35149c858f0ede22a29)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124675/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>