<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/105703/">http://git.reviewboard.kde.org/r/105703/</a>
</td>
</tr>
</table>
<br />
<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 and Thorsten Zachmann.</div>
<div>By Friedrich W. H. Kossebau.</div>
<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 KPrView constructor adapts the actions for navigating in slides only for the text, but not the tooltip and whatsthis property.
Visible by default in the slide navigator in the statusbar (no idea how I missed that before, perhaps there was no tooltip set, so the text got reused), but also if navigation actions are added to the toolbar.
Backport to 2.5 would be setting tooltip and whatsthis to QString(), to avoid new strings.
I considered changing the KoPAView constructor to take an additional parameter "KoPageApp::PageType pageType", so it can setup the proper text strings itself, instead of relying on all the subclasses (well, just Stage ATM) to adjust things. But that would be a too big change to backport. So perhaps commit simple fix to 2.5 and moving action texts adaption to KoPAView in master (would come up with a new patch)?</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;">Run Stage and Flow, no more bad tooltip in 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>stage/part/KPrView.cpp <span style="color: grey">(1606dfa)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105703/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>