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