> On Oct. 17, 2011, 1:33 p.m., Thomas Pfeiffer wrote:
> > This is a very good idea for tabs with more than one element in them (e.g. 
> > Text editing, References, Review).
> > 
> > What it does not make sense for are those tabs that contain only one or two 
> > buttons (e.g. zoom and pan). Tabs are used to split up a UI in different 
> > sections, not to change the cursor, so chaning the cursor by clicking a tab 
> > would be unexpected behavior.
> > Just placing a single button in those tabs which only changes the cursor 
> > would make the whole thing slow and cumbersome.
> > 
> > Things like pan and zoom should go to the main toolbar as they don't do 
> > document manipulation anyway. If we want to allow users to hide the toolbar 
> > altogether but still have those tools, we need to find a different place. 
> > But I don't think they are of the same type as things like the text editing 
> > tab.
> >
> 
> C. Boemann wrote:
>     Which is why I've already removed those two tabs. Zooming is already 
> available through the statusbar. This is also what Jaroslaw Staniak suggested.
>     
>     Anyway this is in no way supposed to be-all-end-all of Words ui. But I 
> still think it's a huge step in the right direction
> 
> Thomas Pfeiffer wrote:
>     Great! Sorry if I missed that information.
>     
>     I didn't expect it to be finished and polished yet (that's why I did not 
> comment on the unfinished-looking layout ;)
>     
>     So, as a first - big - step in the right direction: Ship it!

@Boemann, it's good step. Removing is harder than adding, so the process is not 
going to be very fast and I accept it. Let's also note that mockuping is easier 
than implementing and throwing out, I learn that with the new Kexi Modern GUI. 
You have my support.


- Jarosław


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


On Oct. 15, 2011, 4:49 p.m., C. Boemann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102871/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2011, 4:49 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> This UI was conceived during the KDE User Experience sprint in Berlin 2011
> 
> It basically provides a drop-in-replacement for the toolbox + 
> tooloptionsdocker in the form of a single docker containing QToolBox widget.
> 
> The new KoModeBox is used in Words only. All the other applications should be 
> left unchanged.
> 
> The tools have a new extended usage of the Type attribute. See the 
> KoToolFactoryBase.h changes for description. It should be compatible with the 
> krita usage.
> The only visible changes outside Words would probably be that the gradient, 
> pattern,calligraphy tools end up in a separate section of the toolbox.
> 
> 
> Diffs
> -----
> 
>   karbon/plugins/tools/CalligraphyTool/KarbonCalligraphyToolFactory.cpp 
> cb1c092 
>   karbon/plugins/tools/KarbonGradientToolFactory.cpp 786d98a 
>   karbon/plugins/tools/KarbonPatternToolFactory.cpp 1ec250fa 
>   karbon/plugins/tools/KarbonPencilToolFactory.cpp 4fd1105 
>   karbon/plugins/tools/filterEffectTool/KarbonFilterEffectsToolFactory.cpp 
> c30f1be 
>   libs/flake/KoToolFactoryBase.h 73f2534 
>   libs/flake/KoToolManager.cpp 3bcae08 
>   libs/main/CMakeLists.txt 9539a5a 
>   libs/main/KoDockerManager.h 7e44455 
>   libs/main/KoDockerManager.cpp 7f5b17d 
>   libs/main/KoModeBox.cpp PRE-CREATION 
>   libs/main/KoModeBoxFactory.h PRE-CREATION 
>   libs/main/KoModeBoxFactory.cpp PRE-CREATION 
>   libs/main/KoModeBox_p.h PRE-CREATION 
>   libs/main/KoToolBox.cpp ede46d4 
>   plugins/textshape/ReferencesToolFactory.cpp 096e602 
>   plugins/textshape/ReviewToolFactory.cpp bbd6452 
>   plugins/textshape/TextToolFactory.cpp 438846c 
>   words/part/KWGui.cpp 69d309b 
> 
> Diff: http://git.reviewboard.kde.org/r/102871/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> screentshot of the new ui
>   http://git.reviewboard.kde.org/r/102871/s/306/
> toolbox in karbon
>   http://git.reviewboard.kde.org/r/102871/s/307/
> toolbox in krita
>   http://git.reviewboard.kde.org/r/102871/s/308/
> toolbox in stage
>   http://git.reviewboard.kde.org/r/102871/s/309/
> 
> 
> Thanks,
> 
> C. Boemann
> 
>

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to