Style Update #21
Merged
Style Update #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- add a CSS transition - thicken border and update color
- current color shceme was bugging me - using lichess colors for now, will likely have optional theme to use these later
- only show pointer when hovering over selectable pieces
- outline looks better but isn't supported on mobile - using box-shadow temporarily
- looks better on mobile if on one line
- c/o google fonts - material icons - https://fonts.google.com
- more cleanup tomorrow
- simplify header buttons to only have menu - simplify styling so we can handle more resolutions - board no longer has padding on left and right - i still hate the modal design
- single click on title for now - might make more difficult to see later?
- remove border and rounding on message when screen is smol
- don't round board when window small enough that board hits the edge
- I don't like it, but I don't hate it
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #15
closes #18