D12918: Add filmstrip embed configuration

Thomas Carmichael noreply at phabricator.kde.org
Sat May 26 17:04:29 BST 2018


tcarmichael updated this revision to Diff 34923.
tcarmichael added a comment.


  Updated to reflect changes requested by @elvisangelaccio
  
  The extra `settings->load();` indeed appears unnecessary since it appears to work fine without it.

REPOSITORY
  R343 FFmpeg Thumbnailers

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12918?vs=34260&id=34923

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

AFFECTED FILES
  CMakeLists.txt
  ffmpegthumbnailer.cpp
  ffmpegthumbnailer.h
  ffmpegthumbnailersettings5.kcfg
  ffmpegthumbnailersettings5.kcfgc
  ffmpegthumbs.desktop
  tests/CMakeLists.txt
  tests/ffmpegthumbtest.cpp

To: tcarmichael, #dolphin, ngraham
Cc: elvisangelaccio, cfeck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180526/03929f0c/attachment.htm>


More information about the kfm-devel mailing list