Review Request: Take tool actions from the actioncollection instead of disable them
Sven Langkamp
sven.langkamp at gmail.com
Wed Jul 13 20:52:53 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101937/
-----------------------------------------------------------
(Updated July 13, 2011, 7:52 p.m.)
Review request for Calligra.
Changes
-------
Changed patch. Now simply deactivate all action of tool that are not the active tool.
Summary
-------
Currently all tool actions are added at start and then disabled and enabled. Bug 276150 shows a problem with that: The actions that are colliding with the actions of the new active tool are disabled on activation and enabled at deactivation. This causes a bug when the first tools activate tool actions for non active tools.
The patch solves the problem by adding the action on tool activation and takeing it from the action collection on deactivation.
This addresses bug 276150.
http://bugs.kde.org/show_bug.cgi?id=276150
Diffs (updated)
-----
libs/flake/KoToolManager.cpp 1807fc4
Diff: http://git.reviewboard.kde.org/r/101937/diff
Testing
-------
Works with steps from bug 276150
Thanks,
Sven
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110713/1baae2b0/attachment.htm>
More information about the calligra-devel
mailing list