<table><tr><td style="">rkron created this revision.<br />rkron added reviewers: Plasma: Workspaces, Plasma, broulik.<br />rkron added projects: Plasma, Spectacle.<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/D10146" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>BUG: 385097</p>

<p>This revision further improves the quality of images in notifications that was addressed in revision <a href="https://phabricator.kde.org/D9993" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D9993</a>. When I first looked into this, I tried setting ScaleType on the PreviewJob but it had no effect. I found that the preview images were being scaled to the small vertical size that was requested, and this resulted in my first revision. I should have tried setting ScaleType in addition.</p>

<p>Notification with unpatched libnotificationshelperplugin.so</p>

<p><a href="https://phabricator.kde.org/F5677396" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5677396: before.png</a></p>

<p>Notification with patch applied</p>

<p><a href="https://phabricator.kde.org/F5677397" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5677397: after.png</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Take screenshot with Spectacle with unpatched libnotificationshelperplugin.so  installed and observe the blurry preview image in the notification.</p>

<p>Install the patched libnotificationshelperplugin.so and restart plasmashell.</p>

<p>Take another screenshot with Spectacle and observe the much improved quality of the preview image in the notification.</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/D10146" rel="noreferrer">https://phabricator.kde.org/D10146</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, broulik<br /><strong>Cc: </strong>Plasma, Plasma: Workspaces, plasma-devel, broulik, ngraham, ZrenBot, progwolff, franciscofernandes, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>