Review Request: Rewrote to use tree widgets instead of list widgets

Adam Pigg adam at piggz.co.uk
Wed Jun 20 20:26:53 BST 2012



> On June 20, 2012, 5:04 a.m., Jarosław Staniek wrote:
> > 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

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.


- Adam


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105187/#review14902
-----------------------------------------------------------


On June 11, 2012, 3:22 p.m., Adam Pigg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105187/
> -----------------------------------------------------------
> 
> (Updated June 11, 2012, 3:22 p.m.)
> 
> 
> Review request for Calligra, Jarosław Staniek and Oleg Kukharchuk.
> 
> 
> Description
> -------
> 
> 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
> 
> 
> Diffs
> -----
> 
>   kexi/formeditor/kexiactionselectiondialog.h 7e2d63befc4c4f8ff160bf5ab897024f63f6f590 
>   kexi/formeditor/kexiactionselectiondialog.cpp 15e59d6effe535bfe433d7e274c9e6b49ec79744 
>   kexi/formeditor/kexiactionselectiondialog_p.h 8e2a1fe6820b1e41267ef50fb01aa47d70073cf1 
>   kexi/widget/CMakeLists.txt c4106a3be823a0e54d21ecf5a2bcf24bb8ab246e 
>   kexi/widget/navigator/KexiProjectListView.h 755579bc5c57f187829f7ab3c6c49e204299524e 
>   kexi/widget/navigator/KexiProjectListView.cpp 127ac941e3958c6540593acceab6abd9928bec82 
>   kexi/widget/navigator/KexiProjectListViewItem.h 88a515465ea1c0ed62d23e3ce39bff00a467af19 
>   kexi/widget/navigator/KexiProjectListViewItem.cpp 540aaeec5082bcf2952243ae4a7367a971a0b4a8 
>   kexi/widget/navigator/KexiProjectListView_p.h 9d7b678589937b398d18ef92be676ae382b7210d 
>   kexi/widget/navigator/KexiProjectNavigator.h 009ff78d4d3c9417aa90247da48601406ca06149 
>   kexi/widget/navigator/KexiProjectNavigator.cpp 095422fb149ea76a2c566d66a3412e92c855bddf 
> 
> Diff: http://git.reviewboard.kde.org/r/105187/diff/
> 
> 
> Testing
> -------
> 
> 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.
> 
> 
> Thanks,
> 
> Adam Pigg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120620/e117903e/attachment.htm>


More information about the calligra-devel mailing list