Progress page: remove toggle animation button#34787
Conversation
We don't use the same approach with a button on the Placeholders page.
3a79ca5 to
540f437
Compare
ffoodd
left a comment
There was a problem hiding this comment.
LGTM! Was this done on purpose for accessibility, maybe? Ping @patrickhlauke just in case 😄
nah, i'm pretty sure that was well before my time, and it was there mainly to dynamically demonstrate how adding the class started the animation. no concerns with removing it (as this obeys |
|
|
|
The main reason for having the toggle button is that this animations are not GPU accelerated thus they crippled performance on low-end devices. But since we do not have a button in Placeholders, this makes things consistent . We can bring it back if needed. |
We don't use the same approach with a button on the Placeholders page.
Preview: https://deploy-preview-34787--twbs-bootstrap.netlify.app/docs/5.1/components/progress/#animated-stripes