GNOME Bugzilla – Bug 670086
screenshot: should have a parameter for including pointer
Last modified: 2012-02-14 20:09:02 UTC
The "whole desktop" and "window" methods of the Screenshot interface should have a flag to enable or disable inclusion of the mouse pointer in the image. The area shot is not affected by this, since in that case the pointer is outside of the image area by definition.
Created attachment 207574 [details] [review] screenshot: Add include_cursor parameter Add a boolean parameter to Screenshot and ScreenshotWindow which draws the cursor on the screenshot when set to true. --- This patchs adds that. Works similar as the recorder i.e grab using XFixes and draws on the cairo surface (actually stole parts of the code from there).
Review of attachment 207574 [details] [review]: Can we split out the screenshot stuff to a separate file first?
Created attachment 207581 [details] [review] screenshot: Split into separate file / class Slit the screenshot functionality from ShellGlobal into its own class.
Review of attachment 207581 [details] [review]: "Slit the screenshot functionality" Feeling depressed? Looks fine, otherwise.
Attachment 207574 [details] pushed as b2ec340 - screenshot: Add include_cursor parameter Attachment 207581 [details] pushed as 4516e4c - screenshot: Split into separate file / class