<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/101129/">http://git.reviewboard.kde.org/r/101129/</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, 2011, 3:03 a.m., <b>Thorsten Zachmann</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;">Please don't remove the feature of having space around the slide just to make the new feature work. Try to fix the problem instead of removing the feature.
How about centreing the canvas after loading?
It is not a good idea to store view settings in the document as there can be more the one view. How about save the settings per view. Maybe only save it for the first view but the saving should be done inside the view and not inside the document as when opening only one view is visible. That would also avoid having this extra signal that was added.</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;">Maybe your problem is fixed if the problem is fixed that I wrote about on the mailing list.</pre>
<br />
<p>- Thorsten</p>
<br />
<p>On April 14th, 2011, 10:53 p.m., Paul Mendez 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 Paul Mendez.</div>
<p style="color: grey;"><i>Updated April 14, 2011, 10:53 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;">Add save of zoom factor and mode for normal view and slides sorter. Also eliminate page and width options of zooming in Slides Sorter (these options doesn't make much sense in that view).
Finally I eliminate the white space around the slide because it make the slide lost focus when zooming and when I restore the previous zoom value (the zoom function center the white space too), in addition the white space make too big the default size of shapes when inserted.
(I ask in calligra channel, and the white space was added to have an area for insert things, maybe it could be added in a better way).</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;">build and run</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>kpresenter/part/KPrDocument.h <span style="color: grey">(30bfae0ccee8e4a3775b9629358102bbe85cd6ad)</span></li>
<li>kpresenter/part/KPrDocument.cpp <span style="color: grey">(1d94bb97cbfe0b6922290aa00f38191616feaff4)</span></li>
<li>kpresenter/part/KPrView.h <span style="color: grey">(80b67520be60500ecf11df0827f1d4ee8eeab4e7)</span></li>
<li>kpresenter/part/KPrView.cpp <span style="color: grey">(aef56d9fcb919a314f0095629bea1927916f8059)</span></li>
<li>kpresenter/part/KPrViewModeSlidesSorter.h <span style="color: grey">(ec5f6691072115eb7818f1eb224f6a03fe9bf13c)</span></li>
<li>kpresenter/part/KPrViewModeSlidesSorter.cpp <span style="color: grey">(b4fee3c755aebf65e9bff0cef5516776533ec4a6)</span></li>
<li>libs/kopageapp/KoPAView.cpp <span style="color: grey">(c53edecb309098949f66e9d50238bc60f26f9bee)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101129/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>