After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 700488 - two cursors when screencasting zoom
two cursors when screencasting zoom
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-05-16 21:30 UTC by William Jon McCann
Modified: 2013-05-22 16:34 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
screenshot (76.46 KB, image/png)
2013-05-16 21:30 UTC, William Jon McCann 		  Details
recorder: Hide cursor while magnifier is active (2.33 KB, patch)
2013-05-22 15:53 UTC, Florian Müllner 		committed Details | Review
screenshot: Hide cursor while magnifier is active (2.93 KB, patch)
2013-05-22 16:29 UTC, Florian Müllner 		committed Details | Review

Description William Jon McCann 2013-05-16 21:30:39 UTC 
Created attachment 244469 [details]
screenshot

When screencasting in zoom mode there are two cursors visible.
Comment 1 Florian Müllner 2013-05-22 15:53:20 UTC 
Created attachment 245052 [details] [review]
recorder: Hide cursor while magnifier is active

The magnifier adds its own copy of the system cursor to apply the
expected transformations, so we don't need to add it again in the
recorder; this avoids two different cursors showing up in recordings,
but doesn't address the case where the cursor should not be recorded
at all, but the magnifier adds it anyway.
Comment 2 Jasper St. Pierre (not reading bugmail) 2013-05-22 15:55:49 UTC 
Review of attachment 245052 [details] [review]:

Shouldn't this also be removed for screenshots as well?
Comment 3 Florian Müllner 2013-05-22 16:29:11 UTC 
Created attachment 245065 [details] [review]
screenshot: Hide cursor while magnifier is active

I guess that makes sense, yeah.
Comment 4 Jasper St. Pierre (not reading bugmail) 2013-05-22 16:31:27 UTC 
Review of attachment 245065 [details] [review]:

I was hoping we'd clean this up and have one function implementation for both, but OK.
Comment 5 Jasper St. Pierre (not reading bugmail) 2013-05-22 16:31:37 UTC 
Review of attachment 245052 [details] [review]:

OK.
Comment 6 Florian Müllner 2013-05-22 16:33:56 UTC 
Attachment 245052 [details] pushed as 4fe1360 - recorder: Hide cursor while magnifier is active
Attachment 245065 [details] pushed as b20129c - screenshot: Hide cursor while magnifier is active