-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make the panel indicator a toggle-button #1
Comments
Nice to hear from you, Jimmy! |
Uhm... I can see there's a very similar extension, indeed: https://extensions.gnome.org/extension/2302/daynight/ It uses a toggle-button. |
The extension you recommended is exactly what I wanted to do. It is better than mine 😄! |
Apologies, Only now I understand I was rude in my previous comment... :( You might consider changing the goal of your extension just a bit, maybe. 🤔 Take a look at this, for example: https://github.com/amivaleo/JustAPerfectTheme |
Hi!
This is just a suggestion rather than a bug: the panel indicator might work as a toggle button instead of opening a popover menu. Hence, clicking on the button would either enable or disable the dark theme, according to the previous state of the button. At each of the two states of the button, the indicator could change icon.
Just a suggestion. It would reduce the number of clicks and the number of items added by your extension in the top panel.
If help is needed, I think I can provide an example of such toggle-indicator. I already have a simple extension that works like that, for something completely different. The fact is I am a donkey at coding extensions, so I can give much help apart from providing that example.
💪
The text was updated successfully, but these errors were encountered: