<table><tr><td style="">rkron created this revision.<br />rkron added reviewers: Plasma: Workspaces, Plasma.<br />rkron added projects: Spectacle, Plasma: Workspaces.<br />Restricted Application edited projects, added Plasma; removed Plasma: Workspaces.<br />Restricted Application added a subscriber: plasma-devel.<br />rkron requested review of this revision.
</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/D9993" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch is to improve the quality of images in notifications as reported in <a href="https://bugs.kde.org/show_bug.cgi?id=385097" class="remarkup-link" target="_blank" rel="noreferrer">Bug 385097</a>.</p>

<p>It is still not perfect because KIO::PreviewJob is used to create the thumbnail, and it only produces two sizes, 128px and 256px, so it is likely that a small image will be scaled up, blurring it somewhat.</p>

<p>I took two screenshots of the same Dolphin window. Here is the before, without the patch:<br />
<a href="https://phabricator.kde.org/F5665511" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5665511: Dolphin-screenshot-notification-before.png</a></p>

<p>And the after, with the patch:<br />
<a href="https://phabricator.kde.org/F5665513" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5665513: Dolphin-screenshot-notification-after.png</a></p>

<p>Better, but still not perfect.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Take screenshots with Spectacle with and without the patched libnotificationshelperplugin library and observe the image quality in the notification. plasmashell must be restarted when the library file is changed for it to take effect.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9993" rel="noreferrer">https://phabricator.kde.org/D9993</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/plugin/thumbnailer.cpp</div></div></div><br /><div><strong>To: </strong>rkron, Plasma: Workspaces, Plasma<br /><strong>Cc: </strong>plasma-devel, Plasma: Workspaces, Plasma, ZrenBot, gassaf, ngraham, progwolff, franciscofernandes, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>