Review Request: Add config option to make grid shown behind the shapes in kopageapp

Jarosław Staniek staniek at kde.org
Wed Dec 21 07:13:44 GMT 2011


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


I am surprised this detail hits the eyes of user. Guys, we're again cluttering Calligra. Make it hidden deep in advanced options or better keep in for these that want to edit rc files.

If you cannot decide what setting is better, why user would want to do that? Make defaults per calligra apps - if Krita/Flow differences are needed - fine. But now the burden is moved on the poor user. And why Krita is professional tool, Flow should is not aimed at tweakers.

See http://www.joelonsoftware.com/items/2006/11/21.html _again_ in your free time :)


- Jarosław Staniek


On Dec. 21, 2011, 4:36 a.m., Yue Liu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103483/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2011, 4:36 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> Add config option to make grid shown behind the shapes in kopageapp, one new option added in Settings -> Configure -> Grid.
> 
> 
> Diffs
> -----
> 
>   libs/kopageapp/KoPAViewModeNormal.cpp 7402f3f 
>   libs/main/KoGridData.h 296c812 
>   libs/main/KoGridData.cpp f37d36f 
>   libs/main/config/KoConfigGridPage.cpp a3e1d6f 
> 
> Diff: http://git.reviewboard.kde.org/r/103483/diff/diff
> 
> 
> Testing
> -------
> 
> works well, no problem found.
> 
> 
> Thanks,
> 
> Yue Liu
> 
>

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


More information about the calligra-devel mailing list