Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@istvan-derda
Copy link

Heya, running fedora silverblue 38 with Gnome 44 and noticed that the extension causes the following warnings from gnome-shell when activating it and when it updates the remaining time:

$ journalctl -f /usr/bin/gnome-shell
Mar 12 16:16:07 fedora gnome-shell[1332]: Trying to set label on QuickToggle. Use title instead.

This patch fixes that.

@istvan-derda
Copy link
Author

I just realized that this would break gnome 43 and lower support. What is your policy for that? Support the latest version only? Fork for gnome 44 support? Or is there a smart way to check the gnome version at runtime and support them both? :)

@pomoke
Copy link
Owner

pomoke commented Mar 14, 2023

Regression is not acceptable as this may cause problems for existing users, and mainstream version of gnome-shell is 43 currently. GNOME 44 is mainly in experimental version currently.

I personally prefer checking GNOME version, if eliminating warning is important. But GNOME extension interface is well known for its unstablity, and even break existing extensions from time to time. Will this warning become error or make no effect in future?

@istvan-derda
Copy link
Author

Heya, the field is deprecated, so it will likely be removed in the future, I guess with gnome 45(?) resulting in an error. I'm happy to add a shell version check around it :)

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.

2 participants