Review Request: No longer hijack the escape key to switch to the interaction tool

Thorsten Zachmann t.zachmann at zagge.de
Mon Apr 9 16:50:11 BST 2012


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


I really like the shortcut to use esc to come back to the default tool. However I agree that the functionality should be fixes so that the described behaviour is fixed instead of just removing the feature. Especially in stage and karbon I find that feature really helpful.

- Thorsten Zachmann


On April 9, 2012, 1:31 p.m., Boudewijn Rempt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104524/
> -----------------------------------------------------------
> 
> (Updated April 9, 2012, 1:31 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> The escape key sees a lot of use. It closes the find toolbar, it closes paths in path tools and so on. But no matter what reason the user presses Escape for, it will also enable the Interaction Tool. That means that in words, if you search for a word and press escape to close the find bar, the text tool is disabled. In krita, if you deselect a path, the path tool is deselected.
> 
> I propose to remove this switch, as shown in the patch.
> 
> 
> This addresses bug 297599.
>     http://bugs.kde.org/show_bug.cgi?id=297599
> 
> 
> Diffs
> -----
> 
>   libs/flake/KoToolManager.cpp c121d0f 
> 
> Diff: http://git.reviewboard.kde.org/r/104524/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Boudewijn Rempt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120409/51cdf21a/attachment.htm>


More information about the calligra-devel mailing list