Review Request: change Sheets UI by moving the cell editor out of the cell tool

Inge Wallin inge at lysator.liu.se
Wed Apr 25 23:25:36 BST 2012



> On April 25, 2012, 9:59 p.m., Inge Wallin wrote:
> > This is so awesome!
> > 
> > I found one single bug (but there are probably other small ones). That is that when you open an empty document, the font size select box is empty. When you choose another cell it gets its content.
> > 
> > Also see a small double / qreal issue.
> > 
> > I think this is mature enough to commit, even if I as a non sheets core developer would have preferred some more comments.

Actually I didn't notice that the cell editor could still be accessed even when a cell was is not selected. This behaviour is confusing and should be fixed before the merge.  But I guess that simply disabling the cell editor unless one or more cells are selected is good enough.


- Inge


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


On April 24, 2012, 5:32 a.m., Marijn Kruisselbrink wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104708/
> -----------------------------------------------------------
> 
> (Updated April 24, 2012, 5:32 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> This changes to UI of Calligra Sheets a bit by moving the cell editor widget out of the CellTool and instead have it as a separate docker. On top of that there are several new option widgets for the cell tool that provide common formatting options.
> I've also changed the KoToolDocker a tiny bit, by changing the scroll area in it to not be able to get keyboard focus, at least for these new dockers that is preferred, I'm not sure if there are any other tool option widgets/use cases where you would want the scroll area containing the tool option widgets to be able to get keyboard focus?
> Please test (also available in the tables-uirefactor-mek branch) and see if anything weird is happening since it is a fairly big change.
> 
> 
> Diffs
> -----
> 
>   libs/main/KoToolDocker.cpp 44ce789 
>   sheets/CMakeLists.txt 5355719 
>   sheets/Messages.sh e636ac1 
>   sheets/part/CanvasItem.h 41acefe 
>   sheets/part/CanvasItem.cpp c5ccba3 
>   sheets/part/Factory.cpp fb612da 
>   sheets/part/View.h 5322e43 
>   sheets/part/View.cpp 5c11f98 
>   sheets/ui/ActionOptionWidget.h PRE-CREATION 
>   sheets/ui/ActionOptionWidget.cpp PRE-CREATION 
>   sheets/ui/CellEditorDocker.h PRE-CREATION 
>   sheets/ui/CellEditorDocker.cpp PRE-CREATION 
>   sheets/ui/CellToolBase.h aa3a094 
>   sheets/ui/CellToolBase.cpp 33f2d29 
>   sheets/ui/CellToolBase_p.h 7750a4a 
>   sheets/ui/CellToolBase_p.cpp 0cba586 
>   sheets/ui/CellToolOptionWidget.h c46b7df 
>   sheets/ui/CellToolOptionWidget.cpp 4a1d068 
>   sheets/ui/CellToolOptionWidgets.xml PRE-CREATION 
>   sheets/ui/ExternalEditor.h ab3ee6d 
>   sheets/ui/ExternalEditor.cpp e9f8781 
>   sheets/ui/LocationComboBox.h e666ada 
>   sheets/ui/LocationComboBox.cpp 6215039 
>   sheets/ui/Selection.h baa9cdf 
>   sheets/ui/Selection.cpp 316a2fc 
> 
> Diff: http://git.reviewboard.kde.org/r/104708/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marijn Kruisselbrink
> 
>

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


More information about the calligra-devel mailing list