-
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 d70b5db to 803d9d4 #1854
Conversation
8c10da3
to
dbcfafb
Compare
a8890a6
to
a90dbcc
Compare
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
006ac45
to
acbd92a
Compare
Haven't checked everything in detail, but this first draft seems OK. |
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.
Some things to discuss before approving this one :)
75: .75, | ||
100: 1 | ||
), | ||
), |
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.
[Bootstrap Side] Shouldn't we add some 0 values in here ? Or maybe add state: hover
for text-decoration
? Might be Boosted side only to have links not underlined and underlined on hover ?
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.
LGTM!
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This PR "only" integrates twbs/bootstrap@d70b5db and twbs/bootstrap@803d9d4 to facilitate the review by isolating this new helper.
(integrated the second one to quickly fix the issue with the titles' color in the docs :p)
Live previews
TODO