<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#221146" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D10972#221146</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>In terms of testing with the mediacontroller applet, apart from the Pause problem I found those issues:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Thumbnails for some pictures have incorrect rotation</li>
</ul></div>
</blockquote>

<p>Yes, that is because currently the raw file url is only passed in the metadata. Please see initial question "how to provide thumbnails to the mpris controller via temp files whose url then can be passed in the metadata?" which would allow to handle that, by using instead a properly preprocessed thumbnail. Sadly we can only pass a url in the metadata, so it would need to be some file on the filesystem where the thumbnail is made available. Does Gwenview store the thumbnails in some cache files which could be reused for that?<br />
AFAIK the mpris:artUrl we can use here in the metadata per slide sadly does not specify format or recommended image size, so anything that is fine for Gwenview as preview should be fine with MPRIS as well. One of the things a future MPRIS spec should have better care for.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Rotating images does not reflect in the preview, even after saving</li>
</ul></blockquote>

<p>See above. And for after saving, possibly the media controller UI caches things in memory, or just ignores any rotation settings on loading.<br />
We should assume the average MPRIS controller is not very clever about image formats, so best gets some simple PNG file for the artUrl.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Skipping through pictures with different aspect ratios results in the thumbnail extending over the bounds of the applet</li>
</ul></blockquote>

<p>That seems some Plasma Media controller decision how to do the scaling into the square space used for the "track art". Not specified in MPRIS, so controller decision.</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><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> How expensive is this feature in terms of CPU and DBus traffic for navigating in non-fullscreen browse mode if it were enabled by default for a default Plasma install, i.e. only the systray icon shown?</p></blockquote>

<p>I have no clue about the costs or how to measure that even. But I would not suspect this being expensive, unless people change images and play/slideshow states 1000x per second.</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>