<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/103846/">http://git.reviewboard.kde.org/r/103846/</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 2b6332b6c1b872588d2c01f473e9a6ffc1dfd727 by Sujith Haridasan to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On February 2nd, 2012, 5:29 p.m., Sujith Haridasan 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 and Shantanu Tushar Jha.</div>
<div>By Sujith Haridasan.</div>
<p style="color: grey;"><i>Updated Feb. 2, 2012, 5:29 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;">The following changes were made in this patch:
1) Add the total number of slide number along with the current slide number.
2) If the user views the first slide, hide the previous button(left button).
3) If the user views the last slide, hide the next button( right button).</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;">1) When user launches the CA with the presentation file open, he/she sees the left button disabled if he/she is on the first slide of the presentation. As user moves on to next slide(s) the previous slide button is displayed.
2) When user reaches the last slide, he/she sees the right button disabled. And cannot move to next slide( because its already the last slide). As the user moves to previous slide(s) the next slide button is displayed.
3) Checked the total slide number which is visible throughout the slide.</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>active/qml/PresentationLeftToolbar.qml <span style="color: grey">(bfdcc91)</span></li>
<li>active/qml/PresentationRightToolbar.qml <span style="color: grey">(24ed0b2)</span></li>
<li>active/qml/PresentationTopToolbar.qml <span style="color: grey">(7ebfec9)</span></li>
<li>active/src/CAPresentationHandler.h <span style="color: grey">(1621be5)</span></li>
<li>active/src/CAPresentationHandler.cpp <span style="color: grey">(b5b1aea)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103846/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>