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

Selection menu #377

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from
Draft

Selection menu #377

wants to merge 37 commits into from

Conversation

BF5258
Copy link

@BF5258 BF5258 commented Dec 18, 2024

Adds selection menu with toggle switches to disable dragging pieces and premoves.
Relates to #22 and #328
Don't merge until after #350

To do

How do you want the menu to look?

Possible ideas

  • slider animation
  • sound effect when switch flicked

BF5258 and others added 30 commits November 21, 2024 07:27
@BF5258
Copy link
Author

BF5258 commented Dec 18, 2024

This is what the menu looks like currently:
image
Any suggestions?

@Naviary2
Copy link
Member

Very nice!

An animation would be nice, as most other websites have that.
Also, perhaps add a border? To make the edge stand out more? They feel kinda sharp.
Then I'd shrink the buttons a bit.

Is there a color difference right now between the on and off states? I think it would be cool if the on-state color matched your color theme! For inspiration:

Screenshot 2024-12-18 at 12 33 04 AM

@BF5258
Copy link
Author

BF5258 commented Dec 18, 2024

image

@Naviary2
Copy link
Member

That looks really good!

@BF5258
Copy link
Author

BF5258 commented Dec 19, 2024

There is a thin line of white pixels between the colored area and the border. Is it due to rounding error? How can I fix it?

image

@Naviary2
Copy link
Member

It just looks like its an issue with using outline. Could we get it work work with a border instead?

@Naviary2
Copy link
Member

Screenshot 2024-12-20 at 2 45 18 AM Screenshot 2024-12-20 at 2 45 28 AM Perhaps a white button would look better for our light theme. Similar to these. The off-state background color could be a tad darker tho imo

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