D19782: Allow dolphin to auto-play previewed media file, click on preview to play/pause videos or audio

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a comment.


  This looks fantastic to me. It works great, and the code change seems sane. Let's wait for @elvisangelaccio's review now.

INLINE COMMENTS

> informationpanelcontent.h:86
> +     * Set the auto play media mode for the file previewed
> +     * Eventualy starting media playback when turning it on
> +     * But not stopping it when turning it off

typo: "eventually"

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D19782

To: meven, #dolphin, elvisangelaccio, ngraham
Cc: ngraham, broulik, kfm-devel, pekkah, fprice, fbampaloukas, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190620/ee844871/attachment.htm>