[effects] Support capture cursor image in Screenshot effect
ClosedPublic

Authored by graesslin on Oct 20 2016, 6:32 AM.

Details

Summary

The screenshot methods to blit the screen so far did not capture the
cursor image. But sometimes it is useful to also capture the cursor
image.

This change extends the exported methods to take an optional bool
argument whether to capture the screen. The default is not changed and
still not captures the cursor.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 7559.Oct 20 2016, 6:32 AM
graesslin retitled this revision from to [effects] Support capture cursor image in Screenshot effect.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added projects: Plasma on Wayland, KWin. · View Herald TranscriptOct 20 2016, 6:32 AM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript

Multi-screen screenshot with the cursor image captured.

hein accepted this revision.Nov 7 2016, 9:42 AM
hein added a reviewer: hein.
This revision is now accepted and ready to land.Nov 7 2016, 9:42 AM
This revision was automatically updated to reflect the committed changes.