<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/104607/">http://git.reviewboard.kde.org/r/104607/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 15th, 2012, 5:51 p.m., <b>C. Boemann</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">well i must admit i wasn't think about BIC</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">but yes and then we don't need the other list function anymore</pre>
<br />
<p>- C.</p>
<br />
<p>On April 15th, 2012, 5:11 p.m., Friedrich W. H. Kossebau 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 Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated April 15, 2012, 5:11 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;">(Bah, initial long text was lost on posting, due to connection/server error... :( )
The context menu of the horizontal ruler does not update itself if the unit type of the document is changed e.g. from the page layout dialog.
This patch fixes it. It adds a subclass of QActionGroup, UnitActionGroup, which connects to the "unit changed" signal of the document and updates the checked state of the actions accordingly. It also handles the triggering of the actions in a central place, instead of having each action object do it, should be more lightweight in the end.
Candidate for backport to 2.4?
For now I made it configurable which units are put in the UnitActionGroup. Should instead the constructor of UnitActionGroup simply always create actions for all these units? ATM there is no other user anyway, could there be one?
An issue, which should be cared for in a separate patch IMHO, is that the order here is different from the order of units e.g. in the unit selector in the page layout dialog. I find it confusing that they are not the same orders. Will write an email to the ML for that next :)</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/main/KoView.cpp <span style="color: grey">(81dafd3)</span></li>
<li>libs/main/KoView_p.h <span style="color: grey">(9b3dff2)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104607/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>