Review Request 108569: Flake ConfigWidgets and user friendly interface

[Jean-Nicolas Artaud]

> On Jan. 24, 2013, 2:48 p.m., Arjen Hiemstra wrote:
> > Quite nice, a good improvement on the existing UI. I have a bunch of comments on some of the details though.

Thank you very much for this precise review.
It will take me some time to correct all these points, but I'm on it, and I'll be back with a nicer UI following your comments.


- Jean-Nicolas


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


On Jan. 24, 2013, 2:12 p.m., Jean-Nicolas Artaud wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108569/
> -----------------------------------------------------------
> 
> (Updated Jan. 24, 2013, 2:12 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> This patch totally changes the design of the tools config widget use. This new way of doing had been discussed with several people (boemann and a UI designer worked 8 hours at the aKademy, and this part is one of the main thing to do. The manipulation of shapes on pages and slides is way easier now. The good widgets appear to you when you select a tool, and you know directly have what you need and want to manipulate the shapes!
> 
> - If you select a tool, it will open the config widgets associated.
> - As you can see on the screenshots, all the tools needs their own widgets.
> - Stroke, Fill and Shadow config widgets are the main one.
> - A same config widget can be used with several tools.
> - Snap Settings had been removed from config widgets and added as a Docker -- the opposite way that the StrokeDocker for instance.
> 
> 
> Diffs
> -----
> 
>   karbon/plugins/tools/CalligraphyTool/KarbonCalligraphyTool.h 7de7a14 
>   karbon/plugins/tools/CalligraphyTool/KarbonCalligraphyTool.cpp 4833d86 
>   krita/CMakeLists.txt 835f90d 
>   libs/basicflakes/tools/KoCreatePathTool.cpp 043ec2e 
>   libs/widgets/KoFillConfigWidget.cpp PRE-CREATION 
>   libs/widgets/KoResourcePopupAction.cpp PRE-CREATION 
>   libs/widgets/KoShadowConfigWidget.cpp e9e9502 
> 
> Diff: http://git.reviewboard.kde.org/r/108569/diff/
> 
> 
> Testing
> -------
> 
> Tested in the flake-configwidgets-boemann branch, on words and stage, with several documents, and different kind of shapes.
> 
> 
> File Attachments
> ----------------
> 
> Default Tools config widgets
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/23/DefaultTools.jpg
> New docker list
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/23/NewDockersList.jpg
> Freehand path drawing tool config widgets
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/23/PathDrawingTool.jpg
> Snap Settings Docker
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/23/SnapSettings.jpg
> 
> 
> Thanks,
> 
> Jean-Nicolas Artaud
> 
>

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