<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/103427/">http://git.reviewboard.kde.org/r/103427/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 26th, 2012, 7:05 p.m., <b>Albert Astals Cid</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;">Could you explain why you decided to put it in a separate toolbar and not in the existing one? A new one gives you the versatibility to put the new minibar in each of the 4 sides of the screen, but obviously you don't want it in the bottom and you don't want it on the left or right side either since it's a huuuuuuuuuuuge was of space there, so you are going to have it on top almost always. If we have it on top almost always, i think it makes sense for it to be part of the existing toolbar, so you can actually put in the order you want, e.g. after the zoom combo.
What do you think?</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;">For exactly that reason (versatility), although now having thought about it I agree that having it at a side won't work and at the bottom is pointless. Having a separate toolbar also would allow it to be floated, although I never myself use any KDE toolbars that way and I'm not sure if anyone else would want to.
</pre>
<br />
<p>- Jonathan</p>
<br />
<p>On January 24th, 2012, 10:51 a.m., Jonathan Marten 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 Okular.</div>
<div>By Jonathan Marten.</div>
<p style="color: grey;"><i>Updated Jan. 24, 2012, 10:51 a.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;">The bug (and duplicates) suggests that the page bar which normally appears at the bottom of the Okular window could be docked in the toolbar, in order to save vertical screen space which is especially useful on a wide screen. This patch implements that.
There can only be one MiniBar in existence, because of the fixed observer ID (which must be unique). So the page tool only appears in the toolbar if the page bar is hidden; if it is shown, then the page number and size label appears there as before. The MiniBar is reparented in Part::slotShowBottomBar() when the page bar is shown or hidden.
The page tool is placed in its own toolbar by default, so that it can be positioned or floated in accordance with the user's preference.
There are GUI changes and a new I18N string, so this change (if accepted) would be targeted at KDE SC 4.8.1 or later.
</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;">Built Okular from master with these changes. Checked operation in both embedded and standalone modes with page bar shown or not, with a variety of PDF files.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=279128">279128</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>core/observer.h <span style="color: grey">(266993c)</span></li>
<li>part-viewermode.rc <span style="color: grey">(dbd8e42)</span></li>
<li>part.h <span style="color: grey">(a4ab8aa)</span></li>
<li>part.cpp <span style="color: grey">(cd0b695)</span></li>
<li>part.rc <span style="color: grey">(33d3829)</span></li>
<li>ui/minibar.h <span style="color: grey">(e91a0fd)</span></li>
<li>ui/minibar.cpp <span style="color: grey">(7e77dec)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103427/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/103427/s/372/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/12/16/okular-pagetool_1_400x100.png" style="border: 1px black solid;" alt="With page bar shown" /></a>
<a href="http://git.reviewboard.kde.org/r/103427/s/373/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/12/16/okular-pagetool_2_400x100.png" style="border: 1px black solid;" alt="With page bar hidden" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>