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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On June 20th, 2012, 5:04 a.m., <b>JarosÅ‚aw Staniek</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;">It's good code for master now.

Two minor things to do: 
- double clicking on an item should used accept the dialog
- proposed addition: move 'Create New Object' action to the top of the 2nd list; otherwise we cannot pick this action when there are no objects of given type on the 2nd list</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;">Fixed 1) but 2) will take some more work/thinking.  The 2nd list is from the project model, shared with the navigator, so its not a case of just adding it to the list.  I will commit to master, as this is how it was before, and we can think of a solution.</pre>
<br />








<p>- Adam</p>


<br />
<p>On June 11th, 2012, 3:22 p.m., Adam Pigg wrote:</p>






<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 Calligra, JarosÅ‚aw Staniek and Oleg Kukharchuk.</div>
<div>By Adam Pigg.</div>


<p style="color: grey;"><i>Updated June 11, 2012, 3:22 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;">Rewrote to use tree widgets instead of list widgets


Fixed ;;


Added copyright


Rewrote all list-items using newly defined roles Fixed visibility of 3rd column without using event filter


Remove unused sorting code


Allow to not set the project model to not searchable

This prevents multiple models being added when not required.

Add back the current form action


Action now created properly and wors on form buttons

Also fixed crash in project navigator

Use the part class correctly


Inital port away from Q3/K3

Not fully functioning yet, but compiles and runs
Unused files removed</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;">Compiles Ok

Dialog displays Ok except 3rd column is initially the wrong size (fixes welcome)

Tested nearly all combinations of action, and that the save/load Ok, and that the actions still work in data mode.</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>kexi/formeditor/kexiactionselectiondialog.h <span style="color: grey">(7e2d63befc4c4f8ff160bf5ab897024f63f6f590)</span></li>

 <li>kexi/formeditor/kexiactionselectiondialog.cpp <span style="color: grey">(15e59d6effe535bfe433d7e274c9e6b49ec79744)</span></li>

 <li>kexi/formeditor/kexiactionselectiondialog_p.h <span style="color: grey">(8e2a1fe6820b1e41267ef50fb01aa47d70073cf1)</span></li>

 <li>kexi/widget/CMakeLists.txt <span style="color: grey">(c4106a3be823a0e54d21ecf5a2bcf24bb8ab246e)</span></li>

 <li>kexi/widget/navigator/KexiProjectListView.h <span style="color: grey">(755579bc5c57f187829f7ab3c6c49e204299524e)</span></li>

 <li>kexi/widget/navigator/KexiProjectListView.cpp <span style="color: grey">(127ac941e3958c6540593acceab6abd9928bec82)</span></li>

 <li>kexi/widget/navigator/KexiProjectListViewItem.h <span style="color: grey">(88a515465ea1c0ed62d23e3ce39bff00a467af19)</span></li>

 <li>kexi/widget/navigator/KexiProjectListViewItem.cpp <span style="color: grey">(540aaeec5082bcf2952243ae4a7367a971a0b4a8)</span></li>

 <li>kexi/widget/navigator/KexiProjectListView_p.h <span style="color: grey">(9d7b678589937b398d18ef92be676ae382b7210d)</span></li>

 <li>kexi/widget/navigator/KexiProjectNavigator.h <span style="color: grey">(009ff78d4d3c9417aa90247da48601406ca06149)</span></li>

 <li>kexi/widget/navigator/KexiProjectNavigator.cpp <span style="color: grey">(095422fb149ea76a2c566d66a3412e92c855bddf)</span></li>

</ul>

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




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








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