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

Make the panel indicator a toggle-button #1

Open
amivaleo opened this issue Nov 6, 2019 · 4 comments
Open

Make the panel indicator a toggle-button #1

amivaleo opened this issue Nov 6, 2019 · 4 comments

Comments

@amivaleo
Copy link

amivaleo commented Nov 6, 2019

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.

💪

@lossurdo
Copy link
Owner

lossurdo commented Nov 6, 2019

Nice to hear from you, Jimmy!
Thanks for your suggestion.
I need to improve my extensions development skills and add that feature.
Keep on touch.

@amivaleo
Copy link
Author

amivaleo commented Nov 6, 2019

Uhm... I can see there's a very similar extension, indeed: https://extensions.gnome.org/extension/2302/daynight/

It uses a toggle-button.
So... Just... Like... Copy&paste? 😅

@lossurdo
Copy link
Owner

lossurdo commented Nov 6, 2019

The extension you recommended is exactly what I wanted to do. It is better than mine 😄!
It would be better for me to fork it and make it better, isn't it?

@amivaleo
Copy link
Author

amivaleo commented Nov 7, 2019

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

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

No branches or pull requests

2 participants