Skip to content
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

Dark mode: revert some changes linked to main temporary .text-* utilities usage before dark mode #2414

Merged

Conversation

julien-deramond
Copy link
Contributor

@julien-deramond julien-deramond commented Dec 14, 2023

Description

This PR tackles the following sub-task identified in #2140:

Check modifications in #2030 (most of the one linked to the text colors must be reverted)

This PR contains the only elements I've found to modify that were extra temporary elements coming from this PR. The dark mode implementation already took care of the majority of them by removing mostly extra .text-* classes.

Live previews

@julien-deramond julien-deramond added the color mode Temporary label to highlight color mode issues label Dec 14, 2023
@julien-deramond julien-deramond changed the base branch from main to main-jd-dark-mode December 14, 2023 07:05
Copy link

netlify bot commented Dec 14, 2023

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 948a647
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/65818dd254438300093340f5
😎 Deploy Preview https://deploy-preview-2414--boosted.netlify.app/docs/5.3/components/navbar
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@julien-deramond julien-deramond changed the title Main jd dark mode revert some color changes Dark mode: revert some changes linked to _main_ temporary .text-* utilities usage before dark mode Dec 14, 2023
Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On spinners.md, shouldn't we reintroduce .text-secondary utility (just thinking if people change the secondary color). We could also add inside a <details><summary> with the Bootstrap doc version. Could be done afterwards.

@julien-deramond julien-deramond changed the title Dark mode: revert some changes linked to _main_ temporary .text-* utilities usage before dark mode Dark mode: revert some changes linked to main temporary .text-* utilities usage before dark mode Dec 14, 2023
@julien-deramond julien-deramond force-pushed the main-jd-dark-mode-revert-some-color-changes branch from d7576d4 to 39e223e Compare December 15, 2023 08:36
@julien-deramond julien-deramond force-pushed the main-jd-dark-mode-revert-some-color-changes branch 2 times, most recently from 3774772 to b382267 Compare December 19, 2023 12:23
@julien-deramond julien-deramond force-pushed the main-jd-dark-mode-revert-some-color-changes branch from b382267 to 948a647 Compare December 19, 2023 12:34
Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@julien-deramond julien-deramond merged commit 925329e into main-jd-dark-mode Dec 20, 2023
@julien-deramond julien-deramond deleted the main-jd-dark-mode-revert-some-color-changes branch December 20, 2023 06:40
julien-deramond added a commit that referenced this pull request Dec 21, 2023
julien-deramond added a commit that referenced this pull request Dec 26, 2023
julien-deramond added a commit that referenced this pull request Dec 27, 2023
julien-deramond added a commit that referenced this pull request Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
color mode Temporary label to highlight color mode issues
Projects
Development

Successfully merging this pull request may close these issues.

2 participants