<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>