vibou.gTile icon indicating copy to clipboard operation
vibou.gTile copied to clipboard

Gnome 3.8.1 compatibility

Open albyrock87 opened this issue 12 years ago • 7 comments

Hello Vibou, thank you for your great extension.

I just upgraded to Shell 3.8.1 and your extension does not work. Here is the crashlog:

    JS ERROR: !!!   WARNING: 'assignment to undeclared variable monitorIdx'
    JS ERROR: !!!   WARNING: file '/home/alby/.local/share/gnome-shell/extensions/gTile@vibou/extension.js' line 215 exception 0 number 156

(gnome-shell:5560): Clutter-WARNING **: Attempting to add actor of type 'StBoxLayout' to a container of type 'StButton', but the actor has already a parent of type 'StButton'.

(gnome-shell:5560): Clutter-WARNING **: Attempting to add actor of type 'StBoxLayout' to a container of type 'StButton', but the actor has already a parent of type 'StButton'.

(gnome-shell:5560): Clutter-WARNING **: Attempting to add actor of type 'StBoxLayout' to a container of type 'StButton', but the actor has already a parent of type 'StButton'.

(gnome-shell:5560): Clutter-WARNING **: Attempting to add actor of type 'StBoxLayout' to a container of type 'StButton', but the actor has already a parent of type 'StButton'.
      JS LOG: Extension "gTile@vibou" had error: Error: second argument to Lang.bind() must be a function, not undefined

Thank you!

albyrock87 avatar May 11 '13 16:05 albyrock87

Dear Vibou, I encounter the same problem like albyrock87. In Gnome Shell 3.8.x, the extension is not working anymore. Even after following the steps as described here (https://github.com/vibou/vibou.gTile/issues/16), gTile does not run.

I hope you find the time to look into this issue. Your gTile extension is in fact a very helpful extension! Thanks.

ghost avatar Jun 02 '13 18:06 ghost

Hi everyone once I will pass to GS 3.8 I ll have a look at it. But I m still on ubuntu 13.4 so I still have 3.6.3.1. If I have some time I ll try it in a VM

vibou avatar Oct 16 '13 08:10 vibou

See #22 for an successful attempt at this.

Bengt avatar Oct 16 '13 11:10 Bengt

#22 worked for me. I recently upgraded to Ubuntu 13.10 and the fact that it broke gTile really bothered me.

Thanks Bengt.

gergalerg avatar Oct 26 '13 07:10 gergalerg

Thanks for the report, @gergalerg. I am glad the code from my pull request works for you.

Bengt avatar Oct 27 '13 22:10 Bengt

I am curious as to why that worked though. Why does the "this" keyword no longer work in the parameters?

gergalerg avatar Oct 28 '13 20:10 gergalerg

Sorry, I have no insight on the specifics of the GNOME shell's javascript-API. I just made the pull request after I got it working for myself.

Bengt avatar Oct 28 '13 23:10 Bengt