<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/108891/">http://git.reviewboard.kde.org/r/108891/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">the change was not made in a vacuum. we also received feedback pointing out problems with the existing approach, which some (inc you) would like to bring back. after weighing factors on both sides, we decided to change the current sorting approach, knowing fully well that no matter what we did someone would be unhappy about that decision :)
this has been explained before. it's even referenced in the bug report you link to. to simply keep bringing this up is, for lack of a better word, lacking respect. in life, sometimes people will respectfully disagree with each other as there is no one answer that can satisfy all but rather a multiplicity of answers each of which only satisfies some. it is better to acknowledge the disagreement than to keep trying to force change by endlessly bringing it up. 2 years on, we all have better things to do with our lives.
so this patch is rejected because:
* it has been thoroughly discussed already
* a configuration option is not acceptable for this as there are already too many such options
* there is a qml port that is pending; all changes to the c++ version of the applet have been suspended
if you wish to have a window list that sorts differently, my suggestion is this:
* help with, or wait for, the qml port
* when it comes out, create a fork of it that implements things the way you wish
"Effectively, we, the users, feel this is a feature regression, breaking a functionality that's worked in previous releases just fine."
i have always wondered who spoke for all of our users. and finally, we've found them!</pre>
<br />
<p>- Aaron J.</p>
<br />
<p>On February 11th, 2013, 8:15 p.m. UTC, Michael Biech wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Michael Biech.</div>
<p style="color: grey;"><i>Updated Feb. 11, 2013, 8:15 p.m.</i></p>
<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;">Dear everyone,
I'd like to draw your attention to the somewhat recent activity that's been happening concerning issue #215231 (https://bugs.kde.org/show_bug.cgi?id=215231 ). Basically it boils down to people strongly voicing their opinions on their preference of a "column first" sorting order for the task bar when the "force rows" setting is enabled.
This issue has come up before 2 years ago (see https://svn.reviewboard.kde.org/r/5776/ ), where the request to implement this was denied. Interestingly enough, the negatively received change was only introduced in the 4.6 release. Effectively, we, the users, feel this is a feature regression, breaking a functionality that's worked in previous releases just fine.
We therefore kindly ask for this issue to be reconsidered for resolution, i.e. bringing back the behaviour from pre 4.6 releases.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="None">215231</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108891/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>