rkron created this revision. rkron added reviewers: Plasma: Workspaces, Plasma, broulik. rkron added projects: Plasma, Spectacle. rkron requested review of this revision.
REVISION SUMMARY BUG: 385097 This revision further improves the quality of images in notifications that was addressed in revision https://phabricator.kde.org/D9993. 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. Notification with unpatched libnotificationshelperplugin.so F5677396: before.png <https://phabricator.kde.org/F5677396> Notification with patch applied F5677397: after.png <https://phabricator.kde.org/F5677397> TEST PLAN Take screenshot with Spectacle with unpatched libnotificationshelperplugin.so installed and observe the blurry preview image in the notification. Install the patched libnotificationshelperplugin.so and restart plasmashell. Take another screenshot with Spectacle and observe the much improved quality of the preview image in the notification. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D10146 AFFECTED FILES applets/notifications/plugin/thumbnailer.cpp To: rkron, #plasma_workspaces, #plasma, broulik Cc: #plasma, #plasma_workspaces, plasma-devel, broulik, ngraham, ZrenBot, progwolff, franciscofernandes, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart