-
Notifications
You must be signed in to change notification settings - Fork 51
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 Support #200
Comments
Really like this & am wondering about the timeline for a 45-capable version? |
I just updated to 45 in Arch via extra-testing...it's working flawlessly---just waiting for this extension to complete my extensions. |
This is good news :) |
Yes, pretty please! This is the best screenshot tool for GNOME. |
JS ERROR: Extension gnome-shell-screenshot@ttll.de: SyntaxError: import declarations may only appear at top level of a module @ resource:///org/gnome/shell/misc/extensionUtils.js:4:0 |
I'm working on updating the extension for Gnome Shell 45. As soon as I get it everything to work I'll create a pull request. |
Okay I think I kinda messed up and edited some generated JS files instead of editing the actual typescript files. I think. I'm not very sure. |
Thank you very much for the update, it works well, although if I leave the session, without closing it, and come back in, it always shows me the settings screen, it's not a big problem, I close it and that's it. |
any news? |
Gnome 45 introduced a bunch of breaking changes, a proper upgrade will take a few months
@oxayotl Thank you, this is still useful for me! I will look into doing a one-time JS release of the extension |
Great, thanks!!! |
@oxayotl I just tested your version and I'm able to take screenshots! Thank you! 😄 For those who might need help: |
@oxayotl Any word on that PR? Or do you have a fork with the changes so I can make a PR? |
@jmaargh I edited the generated javascript instead of the original typescript, so I cannot really make a PR. The files I edited don't exist in the repo. I couldn't figure out how the original typescript files were merged into one and I needed to make changes to that part (the “import” declarations have to be at the top of the file now), so I gave up. |
latest commit on master has support for Gnome 45 new version is pending review on e.g.o happy holidays! 🎄 🎅 |
GNOME 45 is in beta now, Please add support for it to ensure compatibility with the latest version of GNOME and provide users with access to new features and improvements.
The text was updated successfully, but these errors were encountered: