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
Merge dark mode preview branch into 5.3 #1550
Conversation
Enable dark mode preview in 5.3 CSS
…nto 5.3-dark-mode-preview
Also, simplify CSS by using `dcf-site-group` to apply to both children – `dcf-site-affiliation` and `dcf-site-title`.
Also adjust comment indents
Add grays and light cerulean
…preview # Conflicts: # package-lock.json
- Add margin utility to effectively remove the margin-bottom from the form label - Remove the scarlet color class from the search button (It’s already scarlet in light mode and will automatically switch to cream in dark mode.)
…preview # Conflicts: # wdn/templates_5.3/scss/generic/_generic.color-scheme.scss
…preview # Conflicts: # package-lock.json
See sonarcloud bug. Looks like probably a typo. |
It was indeed a typo – thanks to my broken command key. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
Looks good.
I would run npm update dcf
followed by npm install
to ensure we have the latest dcf and the latest npm dependencies are included before merging.
I did again – just in case – and it's already up to date. |
No description provided.