-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gnome 45 compatibility #213
Conversation
The "legacy" branch seems a reasonable accommodation for anyone not yet using gnome 45 - we should probably update the README to make it clear though. There are a couple other updates needed there for similar reasons, i.e. #202, so we could leave that for a later documentation-focused update instead. I'll review and test this in a bit more detail at some point this/next week, but having had a brief look at the changes they seem good/reasonable! |
yeah good point, the readme is a bit dated. I opened PR #214 to address these issues, it might need to be rebased before merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good! I need to test this in a Gnome VM asap, then will approve review if there are no issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Appears to work as expected in test VM (Ubuntu 23.10.1, both Wayland/X11)!
This PR contains the changes to imports required for Gnome 45. Since it breaks backwards compatibility, I created a new "legacy" branch from master before this PR that contains the old code compatible with versions 40-44.
I tried to keep the changes minimal, but it was still necessary to change every JS file, since all imports had to be updated.
This addresses issue #212.