-
Notifications
You must be signed in to change notification settings - Fork 921
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
[UG] Refer to Bootstrap 5 doc pages and files #1844
Conversation
[_variables]: https://github.com/google/docsy/blob/main/assets/scss/_variables.scss | ||
[v4-dev/scss/_variables.scss]: https://github.com/twbs/bootstrap/blob/v4-dev/scss/_variables.scss | ||
[Docsy uses Bootstrap 5]: /blog/2023/bootstrap-5-migration/ | ||
[Variable defaults]: https://getbootstrap.com/docs/4.1/getting-started/theming/#variable-defaults |
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.
Is that the related 5.3 page? https://getbootstrap.com/docs/5.3/customize/css-variables/
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.
Oh thanks! I was clearly zip'n too quickly through this one. I'll fix it soon.
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.
Almost, the link we want is https://getbootstrap.com/docs/5.3/customize/sass/#variable-defaults.
Fixed.
5ca9bb6
to
7b662aa
Compare
7b662aa
to
580aa75
Compare
@LisaFC - comments are still welcome on this PR. |
Looks good to me! |
Preview: https://deploy-preview-1844--docsydocs.netlify.app/docs/adding-content/lookandfeel/