rjanja / desktop-capture

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

some JS warning #74

Closed NikoKrause closed 6 years ago

NikoKrause commented 7 years ago

I see some JS warnings in xsession-errors:

Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/applet.js 433]: reference to undefined property this._uuid

and

Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/screenshot.js 398]: Too many arguments to method Clutter.Container.add_actor: expected 1, got 2
Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/screenshot.js 399]: Too many arguments to method Clutter.Container.add_actor: expected 1, got 2
Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/screenshot.js 400]: Too many arguments to method Clutter.Container.add_actor: expected 1, got 2
Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/screenshot.js 401]: Too many arguments to method Clutter.Container.add_actor: expected 1, got 2
Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/screenshot.js 1109]: reference to undefined property event.get_source(...).style_class
Cjs-Message: JS WARNING: [~/.local/share/cinnamon/applets/capture@rjanja/3.2/applet.js 1306]: Too many arguments to method St.TextureCache.load_uri_sync: expected 4, got 5
rjanja commented 7 years ago

Which version of Cinnamon are you running? I'm not seeing the latter group of warnings in 3.2, so I suspect some Clutter method signatures have changed and we'll need a new multi-version target.

NikoKrause commented 7 years ago

I use Cinnamon 3.4. But I get those only when actually using the applet.

I try to figure out the steps to reproduce them. I'll let you know, when I have more info.

NikoKrause commented 7 years ago

So looking at xsession-errors here:

I get those warnings, when taking screenshots with "Area". I don't think you need to add a new multi-version target for that.