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

Chore: merge from ce11411 to f7426c0 #2336

Merged
merged 4 commits into from
Oct 24, 2023

Conversation

julien-deramond
Copy link
Contributor

@julien-deramond julien-deramond commented Oct 24, 2023

@julien-deramond julien-deramond added v5 chore merge Sync with Bootstrap labels Oct 24, 2023
@netlify
Copy link

netlify bot commented Oct 24, 2023

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit e744afa
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/6537e7fb5d7cd000086f8d6d
😎 Deploy Preview https://deploy-preview-2336--boosted.netlify.app/docs/5.3/components/badge
📱 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 marked this pull request as ready for review October 24, 2023 11:54
@@ -192,7 +192,7 @@ An additional navbar (with text or icon items) can be added on the right of the
</svg>
<span class="visually-hidden">Basket</span>
<span class="position-relative align-self-start">
<span class="badge bg-info rounded-pill position-absolute top-0 fs-6 text-white translate-middle">
<span class="badge text-bg-info rounded-pill position-absolute top-0 fs-6 text-white translate-middle">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's no impact here but we will need to define in the dark mode implementation whether .text-bg-info should render the text color in white automatically or black. For now, .text-white is still needed.

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.

One comment to tackle and we're good to go

site/content/docs/5.3/components/badge.md Outdated Show resolved Hide resolved
js/tests/visual/modal.html Show resolved Hide resolved
@julien-deramond julien-deramond changed the title Chore: merge from ce11411 to ?? Chore: merge from ce11411 to f7426c0 Oct 24, 2023
@julien-deramond julien-deramond merged commit 4d39d3c into main Oct 24, 2023
15 checks passed
@julien-deramond julien-deramond deleted the main-jd-chore-merge-from-ce11411 branch October 24, 2023 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore merge Sync with Bootstrap v5
Projects
Development

Successfully merging this pull request may close these issues.

2 participants