Skip to content
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

Porting to GNOME 45 #179

Merged
merged 6 commits into from
Feb 4, 2024
Merged

Porting to GNOME 45 #179

merged 6 commits into from
Feb 4, 2024

Conversation

ff2400t
Copy link
Contributor

@ff2400t ff2400t commented Dec 17, 2023

This PR works fine on my fedora 39 with GNOME 45. It has code for both Gnome 45 and versions before with comments showing which sections work for which version. The code for the older version is commented out. This was based on this comment by @GrylledCheez in #165 (comment)_

I also added settings.ui to the build package which was not included in build.sh script before.

@GrylledCheez
Copy link
Collaborator

Does the preferences window work for you as well? And I don't know how I missed adding the settings.ui

@ff2400t
Copy link
Contributor Author

ff2400t commented Dec 18, 2023

Yep, Preference windows is working for me now.

@GrylledCheez
Copy link
Collaborator

@ff2400t, I'm surprised you didn't have to make any alterations to the keybindings.js

I'll make some small adjustments to your code before I merge it in. But thanks for doing the G45 legwork. I've been dragging my feet until Ubuntu forces it on me in the spring.

change sourceType to module
@GrylledCheez
Copy link
Collaborator

@ff2400t, run eslint and fix your whitespace and quoting issues.
npx eslint . --ext .js --ignore-pattern node_modules/ --fix then perform another commit to your #179 PR

@danieldemus
Copy link

@GrylledCheez Now that the linting is ok, could you merge this and release a version that supports G45. Fedora 39 (and soon 40) have been using it for some time now.

change shell-version to 45
@GrylledCheez GrylledCheez merged commit 1e5d0f1 into Fmstrat:develop Feb 4, 2024
1 check passed
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.

3 participants