-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Add some details for non visible scrollspy elements #36625
Conversation
289f7fb
to
e022aa4
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
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.
Few minor suggestions to enhance this warning.
62cf734
to
6018958
Compare
Please @arcanisgk as your issue, it is out of scope on this PR, try to remove it, and start a new discussion or issue |
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! Let's wait for the review of a native speaker before merging it :)
6018958
to
17b4f44
Compare
On mobile for now, but how's this read?
|
Committed your proposal to save you some time with the "Co-authored-by" info. |
0069f90
to
830e81a
Compare
Co-authored-by: Julien Déramond <julien.deramond@orange.com>
Co-authored-by: Mark Otto <markd.otto@gmail.com>
1118fdd
to
a36872a
Compare
closes #36615
It probably needs proofreading, from a native speaker
Preview : https://deploy-preview-36625--twbs-bootstrap.netlify.app/docs/5.2/components/scrollspy/#how-it-works