<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/125485/">https://git.reviewboard.kde.org/r/125485/</a>
     </td>
    </tr>
   </table>
   <br />





 <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'd prefer if this doesn't go in
In words at least it's on purpose that we keep the text selection. 1) because we wan't it to be selected so other tools can work with it 2) we want it to be there when the text tool is reactivated

some other solution might eventually work like not render the selection if the current tool doesn't want it to be rendered</pre>
 <br />









<p>- Camilla Boemann</p>


<br />
<p>On October 3rd, 2015, 9:56 a.m. UTC, David Faure wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for Calligra and Thorsten Zachmann.</div>
<div>By David Faure.</div>


<p style="color: grey;"><i>Updated Oct. 3, 2015, 9:56 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</div>


<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;">Otherwise text remains selected in Stage after switching to another
shape, which is very unexpected.</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>plugins/textshape/TextTool.cpp <span style="color: grey">(dd9e0df252bb6ab6bcd25bb27c20fb8af29839cc)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/125485/diff/" style="margin-left: 3em;">View Diff</a></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>


 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/10/03/d7bcacc1-e0c0-4021-9055-aa2b77f39a71__text-selection-bug.png">text-selection-bug.png</a></li>

</ul>




  </td>
 </tr>
</table>







  </div>
 </body>
</html>