-
-
Notifications
You must be signed in to change notification settings - Fork 227
feat(nitrotype): init #1942
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
base: main
Are you sure you want to change the base?
feat(nitrotype): init #1942
Conversation
To conform more to the catppuccin way of doing things and remove unused code Co-authored-by: uncenter <[email protected]>
…sing from the batch Co-authored-by: uncenter <[email protected]>
…pport the lootbox modal, and support more styling in the customizer
…e, support dropdown links when hovered over, and improve friend online notifications
- update styles to account for UI updates - switch to LESS with standard library
…ype logo on the homepage
Co-authored-by: uncenter <[email protected]>
} | ||
|
||
& when (@themeOnlyRaceBox = 0) { | ||
::selection { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This until L78 is themed by #lib.defaults.
/* color */ | ||
/* the border surrounding a mantle box */ | ||
|
||
@text-shadow: @border; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could these be moved up with the other Less variables? It's a bit difficult to parse considering that they are used prior to being defined.
/* these elements inhert from .table */ | ||
tr, | ||
tbody { | ||
border-color: fade(@crust, 10%); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typically it's preferred to use a different colour for borders, like overlay0 rather than fading existing colours.
/* promotion zone */ | ||
.leagues--promotion-zone.promote { | ||
color: @green; | ||
background: fade(@green, 15%); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could these be green background with base text?
/* race again tooltip */ | ||
.pop--secondary::before { | ||
background-color: @tooltip; | ||
color: @colored-button-text; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't look like it'll contrast well.
/* dash actions (under the race box) */ | ||
.btn--ghost, | ||
.btn--ghost:focus:not(:hover) { | ||
color: fade(@text, 80%); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
80% seems like it won't contrast well enough, even for faded text. 66% makes it a lot more visible.
Co-authored-by: uncenter <[email protected]>
Co-authored-by: WalkQuackBack <[email protected]>
Co-authored-by: WalkQuackBack <[email protected]>
it is no longer needed
Co-authored-by: WalkQuackBack <[email protected]>
- remove options to configure race box theming - theme Gold properly - make race box theming more faithful to original - make text more visible - correct text leader colors - support teachers page
🎉 Theme for Nitro Type 🎉
Nitro Type is a game that lets you improve your typing skills
💬 Additional Comments 💬
🗒 Checklist 🗒
/styles/<name-of-website>
containing the contents of the/template
directory.userstyles.yml
file with information about the new userstyle.
catppuccin.user.css
- all the CSS for the userstyle, based on thetemplate.
preview.webp
- composite image of all four individual flavor screenshots stitched together,generated via Catwalk.