<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://reviewboard.kde.org/r/5078/">http://reviewboard.kde.org/r/5078/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On August 23rd, 2010, 9 a.m., <b>Marco Martin</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;">this very patch appeared here for several times already.
and as usual, the question is: what real value gives over auto hiding the text when there is not enough space?</pre>
 </blockquote>




 <p>On August 23rd, 2010, 9:53 a.m., <b>Markus Slopianka</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 this patch works with the other one that implements launcher support, a Mac OS X-like Dock (AFAIK it&#39;s similar in Win7) can be implemented without the need to get 3rd party widgets.
With a Dock-like setup I wouldn&#39;t want text other than tooltips.</pre>
 </blockquote>





 <p>On August 23rd, 2010, 9:57 a.m., <b>Beat Wolf</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;">i would actually agree on adding it from the feedback i get when i show kde to people used to windows. it&#39;s one of the first things they ask for.</pre>
 </blockquote>





 <p>On August 23rd, 2010, 10:52 p.m., <b>Aaron Seigo</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;">the number of such features that have appeared over the years is immense, and always people ask for those features ... as long as they are new in Windows. there is no point in chasing taillights just to chase taillights. if the idea is a good one, let&#39;s do it; if it isn&#39;t, let&#39;s not.</pre>
 </blockquote>





 <p>On August 24th, 2010, 8:02 a.m., <b>Beat Wolf</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;">sure. i was just mentioning it because i for myself think its a good idea and other people want it, so from my point of view if the patch has a good quality, and it&#39;s actually using a feature that is there anyway, why not have it.
But i see the other point of views too and all have their merit i think.</pre>
 </blockquote>





 <p>On August 24th, 2010, 2:25 p.m., <b>Todd</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;">I think that the question is not so much a question of why we shouldn&#39;t have titles in the task manager, the question is why we need them.  With grouping and the ability in 4.5 to change between grouped windows by clicking on their thumbnail, at least for me titles are just wasted space.  I understand that for people without compositing they are needed, and some people may still prefer them, but personally they don&#39;t contribute anything, and they are huge relative to just the icons.

I agree that the the current configuration interface for the task manager is getting a bit crowded.  However, there is a way to add the feature without increasing the complexity of the dialog.  Currently there are three grouping options: &quot;Do not group&quot;, &quot;Manually&quot;, and &quot;By Program Name&quot;.  There is also an checkox &quot;Only when taskbar is full&quot;.  The problem is that this option is only meaningful in &quot;By Program Name&quot; mode, and in fact the checkbox is disabled when the other two modes.  So I would suggest getting rid of the checkbox and adding a fourth option in the dropdown &quot;When taskbar is full&quot;, or something along those lines. 

Also, since plasma supports multiple categories in the configuration dialog, it may be worthwhile splitting the current options into categories.</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;">&gt; With grouping and the ability in 4.5 to change between grouped windows by clicking on their thumbnail, at least for me titles are just wasted space.

well, i think in this case is really fake &quot;wasted space&quot; because i would agree if the text would let to have less icons in the taskbar.
but since when there is not enough room the text gets disabled automatically, this is a no issue.
without that i really don&#39;t see use cases except making it look like windows</pre>
<br />








<p>- Marco</p>


<br />
<p>On August 22nd, 2010, 1:52 p.m., Björn Ruberg wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Plasma.</div>
<div>By Björn Ruberg.</div>


<p style="color: grey;"><i>Updated 2010-08-22 13:52:33</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;">This patch adds the option to put the taskbar in an icon-only mode - similar as in Windows 7 . This is an much requested feature in bugzilla.
It is fairly simple and just using features already existing in the code, adding an m_showIconOnly member to the layout and the abstractitem plus the adaption of the config ui.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Moved panel around and made sure it works. Looks actually pretty good this icon-only mode!</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="https://bugs.kde.org/show_bug.cgi?id=159480">159480</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/tasksConfig.ui <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/tasks.cpp <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayout.h <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayout.cpp <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttaskitem.h <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttaskitem.cpp <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupitem.h <span style="color: grey">(1166313)</span></li>

 <li>/trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupitem.cpp <span style="color: grey">(1166313)</span></li>

</ul>

<p><a href="http://reviewboard.kde.org/r/5078/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>