-
Notifications
You must be signed in to change notification settings - Fork 54
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
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -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"> |
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.
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.
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.
One comment to tackle and we're good to go
twbs/bootstrap@d58e86e: we dont' have table head/foot in Boosted. We might need to reintroduce them in a separate PR.