Skip to content

Port to GNOME Shell 49#473

Merged
jmmaranan merged 6 commits into
forge-ext:mainfrom
Eskander:49
Sep 21, 2025
Merged

Port to GNOME Shell 49#473
jmmaranan merged 6 commits into
forge-ext:mainfrom
Eskander:49

Conversation

@Eskander

Copy link
Copy Markdown
Contributor

No description provided.

- Replace `Meta.Window.get_maximized()` with `Meta.Window.is_maximized()`.
- Update `Meta.Window.unmaximize()` calls.

https://gjs.guide/extensions/upgrading/gnome-shell-49.html#meta-window
Changes are not backward compatible.

@juarezr juarezr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we maintain compatibility with versions before 49?

  • This way, users with LTC distros will stay covered. These distros don't update in sync with each Gnome release.
  • It also helps to update the extension while using Gnome 48.

Comment thread metadata.json Outdated

@juarezr juarezr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested and didn't find any problem.
Right now, I'm running the version built from this PR.

@juarezr

juarezr commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

The CI is complaining about formatting issues in lib/extension/window.js.
I think this could de fixed with:

npm run format

@juarezr juarezr requested a review from jmmaranan September 16, 2025 22:20
@juarezr juarezr linked an issue Sep 19, 2025 that may be closed by this pull request
@jmmaranan jmmaranan merged commit 29c091c into forge-ext:main Sep 21, 2025
2 checks passed
@Eskander Eskander deleted the 49 branch September 21, 2025 03:03
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.

Bug: Gnome 49 compatibility

3 participants