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