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


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Dolphin, KDE Base Apps and KDE Accessibility.</div>
<div>By Amandeep Singh.</div>


<p style="color: grey;"><i>Updated Aug. 13, 2012, 5:50 a.m.</i></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Thanks for the review. :)

QAccessible class defines static functions & enums for Accessibility. And the QAccessibileInterface must be implemented for a widget to be accessible to accessibility clients like screen-readers, magnifiers, etc.

I added the Accessible Interface for KItemListContainer, KItemListView and KitemListWidgets, to make dolphin accessible. This particular patch makes it work with KMag, which can now track where the keyboard focus is, within the dolphin view.
With some improvements/additions in the Accessibile interface, it will work great with screen-readers as well.

As you suggested, I removed the unnecessary dependency from KItemListSelectionManger, and also tried to remove the commented code and make it compatible with KDE code policies.</pre>
  </td>
 </tr>
</table>




<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;">Added Accessibility Interfaces for Dolphin Views & Widgets, to make it accessible.
2 New files added in dolphin/ src/ kitemviews/ kitemlistviewaccessible.* that contain the three new classes.</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;">Focus-tracking tested with KMag / KWin. </pre>
  </td>
 </tr>
</table>




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

 <li>dolphin/src/CMakeLists.txt <span style="color: grey">(5c1a6da)</span></li>

 <li>dolphin/src/kitemviews/kitemlistcontainer.cpp <span style="color: grey">(5500851)</span></li>

 <li>dolphin/src/kitemviews/kitemlistcontroller.cpp <span style="color: grey">(88f5d9f)</span></li>

 <li>dolphin/src/kitemviews/kitemlistselectionmanager.cpp <span style="color: grey">(383914d)</span></li>

 <li>dolphin/src/kitemviews/kitemlistview.h <span style="color: grey">(5723b9a)</span></li>

 <li>dolphin/src/kitemviews/kitemlistview.cpp <span style="color: grey">(72b3fd8)</span></li>

 <li>dolphin/src/kitemviews/kitemlistviewaccessible.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>dolphin/src/kitemviews/kitemlistviewaccessible.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>dolphin/src/kitemviews/private/kitemlistviewlayouter.h <span style="color: grey">(da5bd1d)</span></li>

 <li>dolphin/src/tests/CMakeLists.txt <span style="color: grey">(3f906d1)</span></li>

</ul>

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




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




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