Skip to content

Port to GNOME 45 #31

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cr7pt0gr4ph7
Copy link

GNOME Shell 45 moved to ESM (ECMAScript modules), which means that it is not backwards compatible with extensions that target GNOME <= 44 (https://gjs.guide/extensions/upgrading/gnome-shell-45.html).

This PR makes all the necessary changes to make this extension work on GNOME 45 (and hopefully later versions as well).

Due to changes to the JS module system, a shell extension
can only be compatible with GNOME <= 44 OR with GNOME >= 45,
so we have to remove compatibility with older versions.

As a sidenote, extensionUtils.initTranslations()
does not have to be called explicitly anymore.
It will be called by the framework instead.

See also:
https://gjs.guide/extensions/upgrading/gnome-shell-45.html
Extension.lookupByURL returns null while loading the
JS files of an extension, and only returns a non-null
value after the constructor of the ExtensionModule
has returned.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant