<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10972">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D10972#220342" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D10972#220342</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/kossebau/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@kossebau</a> Slowly making progress with my review queue…</p></div>
</blockquote>

<p>Okay, looking forward to you reaching this one :)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Did not read all of the (lengthy) discussion yet, but for a head-start, could you give me a short summary on why I see a disabled pause button instead of a stop** button in the mediacontroller applet? (I discovered the context menu to stop the slideshow again only by accident, i.e. not something normal users would find.) – Thanks ;)</p>

<p>**) In the sense of "make it not move anymore", I'll have to think about whether pause or stop would be right.</p></blockquote>

<p>The (old) MPRIS specification has the two concepts "Pause" (as in hold on current position in track) and "Stop" (as in reset to begin of track). With "Pause" being an optionally supported feature per track, which can be queried by a property CanPause.<br />
The maintainers of the Plasma mediacontroller designed the UI though with rather continious (music) players in mind, where one would only toggle between Pause and Play, and thus only need one button for that (both media controller as well as media controller overlay for taskmanager tooltip).</p>

<p>My initial patches proposed to make that toggle button switch between Pause & Play or Stop & Play, depending on current value of CanPause property. Which was not winning over the minds of the maintainer(s). Too still get at least some version of the patches in in time for Plasma 5.12.3, so people can test & use the given patch here for Gwenview already without having to patch Plasma themselves, I removed that part for the middle button in the media controller(s) again and just made the patches fix the usage of the wrong unconditional PlayPause action everywhere.</p>

<p>So this is not the final solution yet and needs more thinking/discussions/design. But at least progress to the old state.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R260 Gwenview</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10972">https://phabricator.kde.org/D10972</a></div></div><br /><div><strong>To: </strong>kossebau, Gwenview<br /><strong>Cc: </strong>mtijink, ngraham, nicolasfella, KDE Connect, rkflx, broulik<br /></div>