[Media Controller] Show track length if available even if seeking isn't possible
ClosedPublic

Authored by broulik on Mar 15 2018, 3:27 PM.

Details

Summary

When seeking isn't possible but the track duration is known, show a progress bar instead of showing nothing.
A progress bar is also much nicer than a disabled slider here as it's then meant as an indicator instead.

Test Plan

Disabled slider:


Progress bar (this patch)

  • Verified that it is not shown when streaming music where length isn't known

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Mar 15 2018, 3:27 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 15 2018, 3:27 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Mar 15 2018, 3:27 PM
Fuchs accepted this revision.Mar 15 2018, 3:30 PM

+1, looks great to me.
Bit of duplicate control, but given there can be one at most, that seems fine to me.

Also progress bar makes sense, since a playing track is progress, so it should even look good if other plasma themes decide to use something completely different looking for that compared to the slider.

This revision is now accepted and ready to land.Mar 15 2018, 3:30 PM
romangg accepted this revision.Mar 15 2018, 3:37 PM
This revision was automatically updated to reflect the committed changes.

Thanks for the quick patch, works nicely with my CanSeek=false gwenview position WIP patch :)