-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improve Styling #227
Open
richpjames
wants to merge
12
commits into
main
Choose a base branch
from
styling
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Improve Styling #227
Commits on Sep 17, 2024
-
We can use these as a rudimentary design system to ensure our styling stays consistent and DRY
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for 814d4fb - Browse repository at this point
Copy the full SHA 814d4fbView commit details -
Implement vars using existing styles
We've added the vars, lets use them.
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for d21efe8 - Browse repository at this point
Copy the full SHA d21efe8View commit details -
This ensures that main fills the full visible viewport
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for 627738b - Browse repository at this point
Copy the full SHA 627738bView commit details -
This adds a simple column layout with the gameplay area (2/3rds of viewport) to the left and a column displaying meta information to the right (1/3 of viewport).
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for d3fb44d - Browse repository at this point
Copy the full SHA d3fb44dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2d33ba9 - Browse repository at this point
Copy the full SHA 2d33ba9View commit details -
Move player list into Players & Scores column
We also re-add the list styling as I think the bullet points add clarity
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for 24e982f - Browse repository at this point
Copy the full SHA 24e982fView commit details -
Configuration menu - View commit details
-
Copy full SHA for b9635f9 - Browse repository at this point
Copy the full SHA b9635f9View commit details -
Move gameplay components into their appropriate columns
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for dd73161 - Browse repository at this point
Copy the full SHA dd73161View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2014f49 - Browse repository at this point
Copy the full SHA 2014f49View commit details -
This makes it a bit clearer these are labels and not just words
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for 8e18174 - Browse repository at this point
Copy the full SHA 8e18174View commit details -
This may make more sense in the colour-card.css file but as it was styling the top level component I put it in the general css file. I think it's fine that the colour-cards file doesn't use the CSS vars as it's a seperate component, but it makes sense for all the components in this file to.
rich committedSep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for 29ba588 - Browse repository at this point
Copy the full SHA 29ba588View commit details -
rich committed
Sep 17, 2024 Configuration menu - View commit details
-
Copy full SHA for fe2934e - Browse repository at this point
Copy the full SHA fe2934eView commit details
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.