<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/101468/">http://git.reviewboard.kde.org/r/101468/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I think the patch should be committed as it fixes a very big annoyance like the following error message when pressing Ctrl+A in the text shape.
The key sequence 'Ctrl+A' is ambiguous. Use 'Configure Shortcuts'
from the 'Settings' menu to solve the ambiguity.
No action will be triggered.
@Casper can you please explain what kind of shortcuts you want to be active event the tool is not active. Please note that is already possible to have actions in the view that are also active when the tool is not active.
And that is the problem Jan is fixing with this patch. It is just that with this patch the shortcut of the tool overwrites the shortcut of the view which is needed as it fixes the above shown error message.
If there is a need for shortcuts in the application that can't be overwritten by tools that is a new feature which can be added later if that is wanted by someone.
However as the shortcuts are not working at all currently if a tool overwrites them and the user gets a very annoying error message I think this patch should be committed as it improves the current situation.</pre>
<br />
<p>- Thorsten</p>
<br />
<p>On May 29th, 2011, 10:17 a.m., Jan Hambrecht wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By Jan Hambrecht.</div>
<p style="color: grey;"><i>Updated May 29, 2011, 10:17 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">At them moment there is a problem with actions from tools which duplicate standard actions in applications. An example are select all /deselect standard actions. There are calligra applications which create these in their view classes to have support to select/deselect shapes. However some tools like the text tool do have these actions too.
When for instance the text tool gets activated, all its actions are enabled which causes the select all action conflict with that of the application (they have the same standard shortcut).
This patch fixes that as it will disable any action from the application during the activation of the tool with an conflicting action. When disabling the tool again, the application action is enabled again.
That way selecting all works in the text tool again and I could add working support for select all/deselect in the artistic text tool too.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested with the text tool, artistic text tool in karbon and stage.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/flake/KoToolManager.cpp <span style="color: grey">(bf68ad9)</span></li>
<li>libs/flake/KoToolManager_p.h <span style="color: grey">(83ea4de)</span></li>
<li>plugins/artistictextshape/ArtisticTextTool.h <span style="color: grey">(e9b6347)</span></li>
<li>plugins/artistictextshape/ArtisticTextTool.cpp <span style="color: grey">(68debef)</span></li>
<li>plugins/textshape/TextTool.cpp <span style="color: grey">(b95e18e)</span></li>
<li>words/part/words.rc <span style="color: grey">(c4aa3f7)</span></li>
<li>words/part/words_readonly.rc <span style="color: grey">(b4abb2a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101468/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>