<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/104928/">http://git.reviewboard.kde.org/r/104928/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 14th, 2012, 5:10 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;">Some observations I made when testing the patch:
o When moving a object out of the screen, the canvas size is not updated no scrollbar added.
The user first needs to zoom or go to a different page so that the correct scrollbar is shown.
o When you change between a slide that has all content on the page and a slide where also some
content is also left of the slide. Then the visible area jumps to the left and it does not show
the full screen any longer.
</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;">Ok that was a weird bug. I knew it was working just before commiting. I must accidentially have move the definition of updatCanvasSi<e to no longer be a slot. I can't recall I did so,but anyway, the fixed both your concerns.</pre>
<br />
<p>- C.</p>
<br />
<p>On May 14th, 2012, 11:42 p.m., C. Boemann 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 C. Boemann.</div>
<p style="color: grey;"><i>Updated May 14, 2012, 11:42 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;">KoPageApp applications Stage and Flow used to have extra space reserved around the page/slide so objects could be put there. However that made it really hard for the user to work with.
This patch changes that so there si no space around the page/slide. But.. instead you can drag objects outside the page/slide without any limit at all</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've tested zooming in and out in Stage normal, notes, and sorter viewmodes, and in Flow.
I've tested that running presentations still work, however I've not tested presenterview.
I've tested moving shapes outside the page/slide in both Stage and Flow.</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/kopageapp/KoPACanvas.cpp <span style="color: grey">(4a4748c)</span></li>
<li>libs/kopageapp/KoPAView.h <span style="color: grey">(01fbdf1)</span></li>
<li>libs/kopageapp/KoPAView.cpp <span style="color: grey">(4d13dfa)</span></li>
<li>libs/kopageapp/KoPAViewModeNormal.cpp <span style="color: grey">(d5aa5f6)</span></li>
<li>libs/widgets/KoZoomController.h <span style="color: grey">(4f4c6a2)</span></li>
<li>libs/widgets/KoZoomController.cpp <span style="color: grey">(e10c5b4)</span></li>
<li>stage/part/KPrViewModeNotes.h <span style="color: grey">(b707770)</span></li>
<li>stage/part/KPrViewModeNotes.cpp <span style="color: grey">(c951ade)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104928/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>