D15863: [MPris Plugin] Support Seek()

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Oct 1 12:27:43 BST 2018


broulik updated this revision to Diff 42652.
broulik added a comment.


  - Check bounds in `SetPosition`
  - Use `>` again since when I move the slider all the way to the end it sets the position to `length` and then would do nothing

REPOSITORY
  R856 Plasma Browser Integration

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15863?vs=42648&id=42652

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

AFFECTED FILES
  host/mprisplugin.cpp

To: broulik, #plasma, davidedmundson, fvogt, nicolasfella
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181001/59b09df0/attachment-0001.html>


More information about the Plasma-devel mailing list