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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 24th, 2013, 1:29 p.m. BRT, <b>Aaron J. 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;">While usually we don't want to have visual elements hiding and disappearing, in this case I wonder if it would make it more evident/obvious what the bottom items are for (and remove the visual clutter when they aren't needed). Yes, that would be rather "web app like" (for lack of a better term), but it may work nicely here.

The phrase "All selected items" .. this might get rather long depending on the translation? Perhaps just i18np("1 item", "%1 items", itemCount)?

It took me a moment to realize what the [<] button was for; ah, yes, to remove all key combinations. Seeing as those can't be set all at once, does it make sense to allow them to all be reset? If it is kept, I'd recommend something more obvious, such as a button with e.g. "No shortcut" </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;">I took some time investigating some UI patterns for bulk operations until I came up with this one (not sure if it's the more appropriate in this case, I only thought it'd nice to keep columns aligned). I'll try hide/show the bulk table accordingly. As for the [<] bulk action I think it could be useful when you have already assigned a bunch of shortcuts and would like to re-assign them from a clean setup. But yes, I can remove it since the main motivation for that is really the bulk setting of visibility.</pre>
<br />










<p>- Sandro</p>


<br />
<p>On January 24th, 2013, 1:11 p.m. BRT, Sandro Andrade 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 kde-workspace.</div>
<div>By Sandro Andrade.</div>


<p style="color: grey;"><i>Updated Jan. 24, 2013, 1:11 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;">In page 'entries', user can select/deselect a specific or all entries and then change visibility and/or reset shortcut for all of them with a single combo selection/clear button click. Individually selecting all entries causes header's checkbox to automatically be checked. After selecting all entries (individually or by clicking in header's checkbox), a single entry deselection causes header's checkbox to automatically be unchecked. General suggestions and, in particular about UI usability, are appreciated.</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>plasma/generic/applets/systemtray/ui/applet.cpp <span style="color: grey">(09482d7)</span></li>

 <li>plasma/generic/applets/systemtray/ui/autohide.ui <span style="color: grey">(3b6efff)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions1.png">Initial setup</a></li>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions2.png">Setting all visibilities to 'Auto'</a></li>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions3.png">All visibilities adjusted to 'Auto'</a></li>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/01/24/systemtray-bulkactions4.png">Setting two specific item visibilities to 'Always Visible'</a></li>

</ul>





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








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