<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;">sure, let's see how it plays out, and i'll tell if it spoils my needs</pre>
<br />
<p>- Casper</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>