<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/102863/">http://git.reviewboard.kde.org/r/102863/</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;">Ship It!</pre>
 <br />







<p>- mojtaba</p>


<br />
<p>On October 26th, 2011, 6:30 p.m., Inge Wallin 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 Inge Wallin.</div>


<p style="color: grey;"><i>Updated Oct. 26, 2011, 6:30 p.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;">This patch is from Mojtaba Shahi Senobari, and implements deletion of styles in the style manager. The tricky part is that if the deleted is used by some paragraph they have to be assigned another paragraph.

This is Moji's first patch here at the reviewboard.  Be gentle. :)</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;">I tested with a number of combinations of paragraph styles, and it seems to work.</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/kotext/styles/KoParagraphStyle.h <span style="color: grey">(3f9f40c)</span></li>

 <li>libs/kotext/styles/KoParagraphStyle.cpp <span style="color: grey">(cdf16a4)</span></li>

 <li>libs/kotext/styles/KoStyleManager.h <span style="color: grey">(ac145f9)</span></li>

 <li>libs/kotext/styles/KoStyleManager.cpp <span style="color: grey">(fbbc4ee)</span></li>

 <li>plugins/textshape/CMakeLists.txt <span style="color: grey">(52fd060)</span></li>

 <li>plugins/textshape/dialogs/StyleManager.h <span style="color: grey">(93247b7)</span></li>

 <li>plugins/textshape/dialogs/StyleManager.cpp <span style="color: grey">(99ed8b8)</span></li>

 <li>plugins/textshape/dialogs/StylesWidget.h <span style="color: grey">(844bed8)</span></li>

 <li>plugins/textshape/dialogs/StylesWidget.cpp <span style="color: grey">(4f31a01)</span></li>

</ul>

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




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








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