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
Make carousel indicators actual buttons (backport to v4) #33166
Conversation
895a256
to
b9df8ff
Compare
Need another approval just in case @rohit2sharma95. |
2975ddc
to
dccbc35
Compare
I marked it as draft; I'm not 100% sure about the CSS changes and specificity since people might override the selectors. |
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.
Temporarily blocking the PR
Setting this to ready for review, but worth deciding if we're going ahead with this backport or scrapping it. |
Make carousel indicators actual buttons
ee12e9b
to
759bc24
Compare
Revisiting this, I think we will have to skip it for the breaking changes in those selectors. So sorry for the wasted effort so long ago! 😓 |
no worries, 5 is where it's at now anyway |
Manual backport of #32661