Review Request: Fix clicking in selection does not update caret

C. Boemann cbr at boemann.dk
Sat Dec 22 17:44:09 GMT 2012


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

Ship it!


yes sure, but could you please change all the initializers to:
 , m_sometihng(xx)

rather than the other way

and ok to backport

- C. Boemann


On Dec. 22, 2012, 4:47 p.m., Pierre Stirnweiss wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107849/
> -----------------------------------------------------------
> 
> (Updated Dec. 22, 2012, 4:47 p.m.)
> 
> 
> Review request for Calligra and C. Boemann.
> 
> 
> Description
> -------
> 
> When clicking inside a selection in the TextTool, the caret does not update to the new position. Seems to be linked to the uninitialized m_drag (don't ask why). It is a good idea to initialize anyway.
> 
> Ok to backport?
> 
> 
> Diffs
> -----
> 
>   plugins/textshape/TextTool.cpp 24f50e2 
> 
> Diff: http://git.reviewboard.kde.org/r/107849/diff/
> 
> 
> Testing
> -------
> 
> Now it works for me ;)
> 
> 
> Thanks,
> 
> Pierre Stirnweiss
> 
>

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


More information about the calligra-devel mailing list