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

Gnome Shell 40 Port #280

Merged
merged 6 commits into from Apr 15, 2021
Merged

Gnome Shell 40 Port #280

merged 6 commits into from Apr 15, 2021

Conversation

jrahmatzadeh
Copy link
Contributor

extension.js

Removed the deprecated actor for ClipboardIndicator.

prefs.js

  • Gtk.accelerator_parse() now returns first element as ok.
  • show_all() no longer works and needed on GTK4.
  • Instead of HBox use Gtk.Box.
  • margin no longer supported in GTK4 and replaced with more detailed margin properties.

metadata.json

Removed "3.38" and added "40.0" to the shell version.

Works on GNOME Shell +3.36
actor is deprecated for ClipboardIndicator
the first returned element of accelerator_parse is ok
@frandieguez
Copy link

Hope we get this PR merged. I have a similar PR from some days ago but I don't know if it was enough to be merged... #277

@jrahmatzadeh
Copy link
Contributor Author

@frandieguez Sorry, didn't see that one. I'm just going over all popular extensions and porting them.

btw, we don't need show_all() and we don't need to check that with typeof. dropping that is safe for GNOME Shell +3.36.

Anyway, we shouldn't add 3.36 and 3.38 support for this one since I've mentioned about Gtk.accelerator_parse(). I've added this one to the port guide too and soon will be public.

If supporting older versions is the end, I can add version checking.

@prateekmedia
Copy link

prateekmedia commented Mar 28, 2021

@justperfection Please also get rid of dropdown button as it looks ugly with gnome 40 design and also because they removed it from the power and network strip.

@jrahmatzadeh
Copy link
Contributor Author

@prateekmedia the prefs has dropdown visibility option and works on GNOME Shell 40.

metadata.json Outdated Show resolved Hide resolved
@prateekmedia
Copy link

prateekmedia commented Mar 29, 2021

@justperfection I don't know enough, but I had done the same i.e. 40, 41, 42 for my extension without .0 , but it is showing incompatible in EGO with gnome 40. Here's the link
https://extensions.gnome.org/review/23140

So I reverted back to 40.0 and so on...

@jrahmatzadeh
Copy link
Contributor Author

jrahmatzadeh commented Mar 29, 2021

@prateekmedia "40" is the accepted version since yesterday. This is only applies to the new submitted extensions and not pending for review extensions. I also change the official port guide.

@rmader
Copy link
Contributor

rmader commented Apr 1, 2021

@prateekmedia the prefs has dropdown visibility option and works on GNOME Shell 40.

@justperfection could you change the default behaviour to disabled, in order to match default 40? IMO it would be even cleaner to just drop the future, but that can probably wait a bit more.

Change drop down arrow default to match GNOME Shell 40
@jrahmatzadeh
Copy link
Contributor Author

@rmader done!

@abhaypatil2000
Copy link

@justperfection it doesn't work for arch linux. I git cloned your repo in extension folder and then restarted the gnome shell. And then opened extensions app and still it shows not compatible with your gnome version. I am using arch linux with gnome 40

@jrahmatzadeh
Copy link
Contributor Author

jrahmatzadeh commented Apr 9, 2021

@abhaypatil2000 because you are not using my gnome-shell-40 branch.

@RuiGuilherme
Copy link

Works on Arch;

Captura de tela de 2021-04-09 03-30-52

Btw, it works fine with my patch for images support. #283

@verbalius
Copy link

tested on Arch and it works without problems. Keyboard shortcuts, all settings and notifications.
image

@rmader
Copy link
Contributor

rmader commented Apr 9, 2021

@Tudmotu would love to see you finding time to land this :)

In case you are busy with other stuff, maybe we could have additional maintainers (I'd be open to volunteer for that, probably others as well) or even move the extension to the officially supported ones (https://gitlab.gnome.org/GNOME/gnome-shell-extensions/) in order to keep it working on new GS versions.

This was referenced Apr 15, 2021
@Tudmotu Tudmotu merged commit 9bc2307 into Tudmotu:master Apr 15, 2021
@Tudmotu
Copy link
Owner

Tudmotu commented Apr 15, 2021

Thank you @justperfection for this PR! 👏 😃
This is very appreciated. Thank you. 🙏

@Tudmotu
Copy link
Owner

Tudmotu commented Apr 15, 2021

Uploaded to e.g.o, will hopefully be approved in the next couple of days.

@jrahmatzadeh jrahmatzadeh deleted the gnome-shell-40 branch April 15, 2021 12:04
@Tudmotu
Copy link
Owner

Tudmotu commented Apr 16, 2021

New version is now approved on e.g.o :)
Thanks everyone for pushing this fix

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.

None yet

8 participants