<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/105366/">http://git.reviewboard.kde.org/r/105366/</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;">I think we should use libs/kundo2/ as we do in the form designer.</pre>
 <br />







<p>- JarosÅ‚aw</p>


<br />
<p>On June 26th, 2012, 9:12 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 and JarosÅ‚aw Staniek.</div>
<div>By Adam Pigg.</div>


<p style="color: grey;"><i>Updated June 26, 2012, 9:12 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;">Merge branch 'master' into kexi-q3port-piggz

Conflicts:
        plugins/chartshape/dialogs/FontEditorDialog.h

Compiles now


porting++


Started port to QUndoGroup


Removed include


Fixed select on double-click

Also removed unused file

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;">Opened an existing table, changed name, and a crash occurs because it is unable to find the undo action in the collection (i think)</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/plugins/tables/CMakeLists.txt <span style="color: grey">(a29b2b944d071149f92d46b8ecc09885d4604035)</span></li>

 <li>kexi/plugins/tables/kexilookupcolumnpage.cpp <span style="color: grey">(4c459c41162400c3427cd04ef11a40f7b553f9a9)</span></li>

 <li>kexi/plugins/tables/kexitabledesignercommands.h <span style="color: grey">(e906388055f508f0316b4fee3fdccb2e4379b8d8)</span></li>

 <li>kexi/plugins/tables/kexitabledesignercommands.cpp <span style="color: grey">(89bd10f939f17083cac0e9fa83c6c628fb555009)</span></li>

 <li>kexi/plugins/tables/kexitabledesignerview.h <span style="color: grey">(04710ffda0808563ebbfd73ecbbcddee72224267)</span></li>

 <li>kexi/plugins/tables/kexitabledesignerview.cpp <span style="color: grey">(8ec956d73ca7874558ac5e3a1d5bd68a18c3c815)</span></li>

 <li>kexi/plugins/tables/kexitabledesignerview_p.h <span style="color: grey">(8f2744fc52d5902f89894bef107109ff09909236)</span></li>

 <li>kexi/plugins/tables/kexitabledesignerview_p.cpp <span style="color: grey">(49af8fcfc7be103a841cf7a5ffd8abbd75e694bc)</span></li>

</ul>

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




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








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