rjanja / desktop-capture

Screenshot and screencasting tools for Cinnamon.
40 stars 11 forks source link

Black box around the mouse cursor #62

Closed crydalch closed 7 years ago

crydalch commented 7 years ago

I've noticed the mouse cursor has a black box around it; is this an issue with Cinnamon, or with the desktop-capture applet? This is Mint 18.1, Cinnamon 3.2.7.

screenshot-area-2017-01-09-210930

Thanks

crydalch commented 7 years ago

FWIW, with maim I don't see the blackbox:

foobar2

rjanja commented 7 years ago

Hi, thank you for the feedback. What is maim? Does the cursor show a black box for you even when you don't use this applet to take a screenshot?

crydalch commented 7 years ago

Hey! This is maim:https://github.com/naelstrof/maim

I don't see the box if I use gnome-screenshot -p from the commandline. How do I take a native cinnamon screenshot?

rjanja commented 7 years ago

maim and gnome-screenshot both use X to take the screenshots. desktop-capture applet uses cinnamon for its screenshots.

I have been able to reproduce this when capturing the mouse cursor. This is either a Cinnamon or Muffin problem I think.

You can instruct Cinnamon to take a full screenshot using the command below, to reproduce what you've described above:

dbus-send --session --type=method_call --dest=org.Cinnamon /org/Cinnamon org.Cinnamon.Screenshot boolean:true boolean:true string:/tmp/test.png
NikoKrause commented 7 years ago

I can reproduce it. screenshot-cinnamon-2017-01-25-102748

Also I tried to take a screenshot of the whole desktop. The screen started flickering like crazy and I get a bunch of the warning below:

(cinnamon:5132): Cjs-WARNING **: JS ERROR: GdkPixbuf.PixbufError: Unrecognized image file format
MyApplet.prototype.maybeSendNotification@/home/krause/.local/share/cinnamon/applets/capture@rjanja/applet.js:1319
MyApplet.prototype.cinnamon_camera_complete@/home/krause/.local/share/cinnamon/applets/capture@rjanja/applet.js:1284
ScreenshotHelper.prototype.runCallback@/home/krause/.local/share/cinnamon/applets/capture@rjanja/screenshot.js:856
ScreenshotHelper.prototype.screenshotScreen/<@/home/krause/.local/share/cinnamon/applets/capture@rjanja/screenshot.js:640
crydalch commented 7 years ago

Thanks guys, logged a ticket against Cinnamon. @rjanja, do you want to close this, or leave it open until it gets fixed upstream? I'm fine either way.

rjanja commented 7 years ago

I think we can close this. While I'd like to help hunt this down in Cinnamon, I just don't have the time these days. Thanks @crydalch

NikoKrause commented 7 years ago

https://github.com/linuxmint/Cinnamon/commit/b9dc41846ddb5aedabf962c20acfc71aec46d5d5