[Okular-devel] Review Request: Improve activation of video playback in Okular

[Tobias Koenig]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104159/
-----------------------------------------------------------

(Updated March 10, 2012, 11:37 a.m.)


Review request for Okular and Albert Astals Cid.


Changes
-------

This updated patch uses show()/hide() instead of the setActive() hack.


Description
-------

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.


Diffs (updated)
-----

  ui/pageview.cpp 344a944 
  ui/pageviewutils.cpp 57e63cf 
  ui/presentationwidget.h e90d21b 
  ui/presentationwidget.cpp bbeb9f5 
  ui/videowidget.h b3ca14e 

Diff: http://git.reviewboard.kde.org/r/104159/diff/


Testing
-------


Thanks,

Tobias Koenig

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20120310/68f4c2bc/attachment.html>