<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/104159/">http://git.reviewboard.kde.org/r/104159/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On March 5th, 2012, 10:34 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;">I don't think i like this much.
 * Who is painting that 'first frame' image? What if there is none?
 * It seems to me that show(); setActive(false); setActive(true); will result in stuff still not being shown, doesn't look right
 * MovieAnnotations are still annotations, why should we not let people change the contents like for the rest of annotations?</pre>
 </blockquote>




 <p>On March 7th, 2012, 1:40 p.m., <b>Tobias Koenig</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;">Hej Albert,

the 'first frame' image is a normal PDF image object that is placed at the same position where the movie annotation is located (this is done internally by the generating app like latex-beamer or ooimpress).
However the image and the movie annotation are not associated/referenced in some way (inside the PDF document), so we can't say "only if there is a first-frame image hide the video widget".
If there is no 'first frame' image, then the area will stay empty, that's consistent with the behaviour of Acroread, so this won't confuse users.

You are right about the show()/setActive(false)/setActive(true) sequence, will fix this.

I just thought that the movie annotation can not really be seen as an annotation because it cannot be changed by the user (unlike a text annotation where the text can be changed). But I won't fight about this one, if you think it should behave like a normal annotation, then I can omit this part of the patch.

Any other objections/suggestions?</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;">Are you sure that for a PDF file without first-frame Adobe shows a blank? Can you please point me to such a file?</pre>
<br />








<p>- Albert</p>


<br />
<p>On March 7th, 2012, 3:07 p.m., Tobias Koenig 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 and Albert Astals Cid.</div>
<div>By Tobias Koenig.</div>


<p style="color: grey;"><i>Updated March 7, 2012, 3:07 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 patch improves the activation of video playback by hiding the VideoWidget until the user has clicked the area where the video is located. This will ensure that a 'first frame' image is visible and won't be covered by a black Phonon::VideoPlayer widget.
Additionally it adds some checks to change the mouse pointer to a different cursor when the video area is entered and it disables the 'edit annotation' dialog for MovieAnnotations.</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>ui/pageview.cpp <span style="color: grey">(344a944)</span></li>

 <li>ui/presentationwidget.h <span style="color: grey">(e90d21b)</span></li>

 <li>ui/presentationwidget.cpp <span style="color: grey">(bbeb9f5)</span></li>

 <li>ui/videowidget.h <span style="color: grey">(b3ca14e)</span></li>

 <li>ui/videowidget.cpp <span style="color: grey">(2a48772)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/104159/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>