-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat(tailwind): setup tailwind themes #4600
Conversation
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.
code review approved, didn't check if it still works
i does not work ( :D yet ) |
|
||
--opacity-background-gradient: 100; | ||
|
||
--backgroud-color-button-primary: var(--color-yellow-500); |
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.
In the InputRef PR, I've fixed this typo: backgroud-*
in the main.css and tailwind config files.
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.
nice
--border-color-button-disabled: var(--color-gray.100); | ||
--border-color-button-disabled-hover: var(--color-gray.100); | ||
--border-color-menu-active: var(--color-orange-500); | ||
--border-color-search-button: #CCCCCC; |
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.
Should these colors be variables? We might need to expand the palettes to account for new colors.
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.
yes but not part of this pr
Quality Gate passedIssues Measures |
What are the main changes you did
Support for multiple themes
styling using tailwind, apply theme using data-theme attr
follow design
light, dark, molgenis, umcg ( aumc = beta )
add config for light
add config for dark
replace 'hard coded' styles with theme styles
add 1 or more demo page to show/evaluate styles
How to test
Checklist