Further improve quality of images in notifications (Bug 385097)
ClosedPublic

Authored by rkron on Jan 27 2018, 4:10 PM.

Details

Summary

BUG: 385097

This revision further improves the quality of images in notifications that was addressed in revision 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

Notification with patch applied

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.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
rkron requested review of this revision.Jan 27 2018, 4:10 PM
rkron created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.Jan 28 2018, 9:18 PM
This revision was automatically updated to reflect the committed changes.

This revision was not accepted when it landed; it landed in state

For benefit of paperwork, it is implicitly accepted by me, I just didn't click the button before merging.

I would have approved too, had you not beat me too it!