> On March 20, 2011, 11:59 a.m., Thorsten Zachmann wrote:
> > Hello Jan,
> > 
> > when testing I found the following:
> > 
> > - After changing the setting it does not take effect before restarting the 
> > application.
> > - In flow and stage somehow the setting is not saved at all.
> > 
> > Is it wanted that the shape is placed with its center at the cursor?
> > 
> > Thorsten
> >

Yes, I noticed this problem too. But I think it is outside of the scope of this 
patch to fix that. (I only added to the code handling the settings which was 
already there).
And placing the shapes with their center at the cursor is intentional. Do you 
think of something different/better?


- Jan


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


On March 19, 2011, 8:30 p.m., Jan Hambrecht wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100891/
> -----------------------------------------------------------
> 
> (Updated March 19, 2011, 8:30 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Summary
> -------
> 
> Patch provides an alternative way to position shapes when pasting them to the 
> document. The new way allows shapes to be positioned at the position of the 
> mouse cursor within the current canvas. Which behaviour is used can be 
> configured via a shared config page.
> 
> 
> This addresses bug 129096.
>     http://bugs.kde.org/show_bug.cgi?id=129096
> 
> 
> Diffs
> -----
> 
>   libs/flake/KoShapePaste.cpp 3cb5d58 
>   libs/main/config/KoConfigMiscPage.cpp 8c1375c 
> 
> Diff: http://git.reviewboard.kde.org/r/100891/diff
> 
> 
> Testing
> -------
> 
> Yes, works like expected.
> 
> 
> Thanks,
> 
> Jan
> 
>

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

Reply via email to