<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/105198/">http://git.reviewboard.kde.org/r/105198/</a>
     </td>
    </tr>
   </table>
   <br />








<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On June 10th, 2012, 6:25 p.m., <b>C. Boemann</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="http://git.reviewboard.kde.org/r/105198/diff/2/?file=67217#file67217line48" style="color: black; font-weight: bold; text-decoration: underline;">libs/textlayout/ListItemsHelper.h</a>
    <span style="font-weight: normal;">

     (Diff revision 2)

    </span>
   </th>
  </tr>
 </thead>

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">struct ListStyleItem {</pre></td>

  </tr>
 </tbody>




 
 



 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">48</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="n">TEXTLAYOUT_EXPORT</span> <span class="n">QString</span> <span class="n">intToNumberingStyle</span><span class="p">(</span><span class="kt">int</span> <span class="n">index</span><span class="p">,</span> <span class="n">KoListStyle</span><span class="o">::</span><span class="n">Style</span> <span class="n">listStyle</span><span class="p">,</span> <span class="n">bool</span> <span class="n">letterSynchronizations</span><span class="p">);</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">wow it seems we have a lot of duplication here with libs/odf/KoOdfNumberDefinition  ::formattedNumber</pre>
 </blockquote>





</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">btw this wasn't meant to raise an issue for your patch - more like a general observation of something we should fix one day.</pre>
<br />




<p>- C.</p>


<br />
<p>On June 10th, 2012, 6:08 p.m., Gopalakrishna Bhat 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 Gopalakrishna Bhat.</div>


<p style="color: grey;"><i>Updated June 10, 2012, 6:08 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 fixes bugs 260083 and 298063 by increasing the display level of the list when the list level is changed either by pressing tab or by using the "Increase Indent" button.</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/commands/ChangeListCommand.cpp <span style="color: grey">(ea180fb)</span></li>

 <li>libs/kotext/styles/KoListLevelProperties.cpp <span style="color: grey">(12085e1)</span></li>

 <li>libs/textlayout/ListItemsHelper.h <span style="color: grey">(c021b68)</span></li>

 <li>libs/textlayout/ListItemsHelper.cpp <span style="color: grey">(d4e409c)</span></li>

 <li>plugins/textshape/TextTool.cpp <span style="color: grey">(dff1fe1)</span></li>

 <li>plugins/textshape/commands/ChangeListLevelCommand.cpp <span style="color: grey">(fdb5833)</span></li>

</ul>

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




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








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