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

Add settings button #541

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

shukenmg
Copy link
Contributor

@shukenmg shukenmg commented Sep 22, 2020

Adds a settings button

Needed to adjust the layout a little bit to add the new button.

#534

@Davidobot
Copy link
Owner

I prefer keeping the UI as clean as possible. Adding more buttons is not the way to do that. The way I would prefer to address the issue posed is to make the cursor change upon hover over the arrow.

@shukenmg
Copy link
Contributor Author

shukenmg commented Sep 22, 2020

@Davidobot

Well, when I first used the software I didn't even see the > it was very obscure, so making the cursor change wouldn't help for readability.

From a user experience standpoint, a much clearer way to see where to open options is needed.

@Davidobot Davidobot changed the title Deals with #534 Add settings button Nov 2, 2020
@cybergrimes
Copy link

I only discovered the > button because I was looking for topics related to power settings.

@cobalt2727
Copy link
Contributor

Please merge this or look into another approach - I've personally helped a lot of people set this program up that didn't even see the tiny arrow until I told them exactly where to look

@mmikeww
Copy link

mmikeww commented Jan 5, 2024

lol just saw the > button due to this PR. i was editing the settings file manually

@cobalt2727
Copy link
Contributor

I have never seen any other piece of software use the > symbol to indicate a settings menu. And even then, you would expect there to actually be a visible button...

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.

5 participants