readingstrip icon indicating copy to clipboard operation
readingstrip copied to clipboard

GNOME 45 support

Open idoric opened this issue 1 year ago • 6 comments

GNOME Shell moved from GJS’ own custom imports system to standard JavaScript modules (ESM). That breaking change require a separate release for 45 and <=44.

https://blogs.gnome.org/shell-dev/2023/09/02/extensions-in-gnome-45/

idoric avatar Sep 21 '23 10:09 idoric

Have a partial port to Gnome 45, https://github.com/sobkas/readingstrip Couldn't port preferences yet.

sobkas avatar Nov 11 '23 15:11 sobkas

I just installed it. At least now I can use it under GNOME 45. Thank you for taking care of this extension.

idoric avatar Nov 12 '23 14:11 idoric

Have a partial port to Gnome 45, https://github.com/sobkas/readingstrip Couldn't port preferences yet.

Is it worth opening a WIP PR here so the maintainer (and others) can chime in (and maybe help complete it?)?

sanjayankur31 avatar Dec 07 '23 12:12 sanjayankur31

This is solved here: https://github.com/lupantano/readingstrip/pull/19/files

Please do test it, folks :)

@sobkas Thanks for porting the extension. I ported it with prefs. Wish I had seen this issue first since you've done lots of work. Please review my PR if you can.

petar-v avatar Mar 16 '24 05:03 petar-v

@petar-v I just tested it quickly, everything seems to work. Well done and thank you. The only improvement I could see is to replace the "show/hide" with a switch.

idoric avatar Mar 16 '24 13:03 idoric

@idoric Thanks so much!

New functionality is beyond the scope of this but I am happy to do that in another PR at some point in time. Otherwise, if you have that implemented, I'll cherry-pick it from you :)

Glad it works, though.

petar-v avatar Mar 16 '24 21:03 petar-v

@petar-v Could I have a clue how to install your updated version? I'm on G46 and need the extension badly

kbdragon avatar May 23 '24 08:05 kbdragon

@kbdragon Extract [email protected] in ~/.local/share/gnome-shell/extensions. You will certainly have to exit and restart your session before you can activate it, for example with Extension Manager.

idoric avatar Jun 09 '24 15:06 idoric

@idoric Thank you!

kbdragon avatar Jun 09 '24 15:06 kbdragon

I merged your code with the latest version of readingstrip. It is definitely buggy, try it.

lupantano avatar Jun 30 '24 05:06 lupantano

It's not my code, it's @petar-v's (see https://github.com/lupantano/readingstrip/issues/17#issuecomment-2001702497). And his version works for me. Maybe the difference is in the run.sh file?

idoric avatar Jun 30 '24 09:06 idoric

now it works on gnome 45 and 46

lupantano avatar Jul 01 '24 04:07 lupantano

Thanks everyone! Just upgraded and works perfectly!

petar-v avatar Jul 03 '24 23:07 petar-v