<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/105180/">http://git.reviewboard.kde.org/r/105180/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 7010a78b8673037d995c331db69630423bb7e9d1 by Friedrich W. H. Kossebau to branch connection-interaction-yue.</pre>
<br />
<p>- Commit</p>
<br />
<p>On June 7th, 2012, 6:14 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 June 7, 2012, 6:14 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;">Currently in Stage/Flow the status bar just has a read-only display for current page number and pages count.
Which also does not tell if the current pages are Master pages or not.
Attached patch fixes that, by turning the label into a display that turns into an editor, like the zoom controller.
Commit log:
It has 2 states:
in first state it displays
* type of slide/page (incl. if Master one)
* the number of the current page/slide
* the number of the page/slide count
in second state it offers control to
* go to first/previous/next/last slide/page
* go by entered number to slide/page
Does not use the icons of the original go* actions, as they look to complex</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;">Used in Flow and Stage.</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/CMakeLists.txt <span style="color: grey">(a353892)</span></li>
<li>libs/kopageapp/KoPAView.h <span style="color: grey">(01fbdf1)</span></li>
<li>libs/kopageapp/KoPAView.cpp <span style="color: grey">(6fccdb0)</span></li>
<li>libs/kopageapp/widgets/KoPageNavigator.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/kopageapp/widgets/KoPageNavigator.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/kopageapp/widgets/KoPageNavigatorButton.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/kopageapp/widgets/KoPageNavigatorButton_p.h <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105180/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/105180/s/600/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/06/07/PageNavigatorDisplay_400x100.png" style="border: 1px black solid;" alt="Display mode" /></a>
<a href="http://git.reviewboard.kde.org/r/105180/s/601/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/06/07/PageNavigatorEdit_400x100.png" style="border: 1px black solid;" alt="Edit mode" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>