fix(Loading): use description prop as SVG title #2421
fix(Loading): use description prop as SVG title #2421
Conversation
/** | ||
* Specify the description for the inline loading text | ||
*/ | ||
iconDescription: PropTypes.string, |
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.
What's the difference between this and description? Can we just use description instead of adding a new prop? Looks like you might have accidentally not updated the comment block
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.
description
is meant to be a node instead of a string, and iconDescription
is for the SVG title in <Loading />
b93281f
to
869cf45
Compare
869cf45
to
d2e6312
Compare
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 👍 - Thanks @emyarod!
@vpicone Would you be able to re-review? Thanks! |
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.
How long are we going to be backporting to v6?
until v8 is released I imagine |
Closes carbon-design-system/carbon#4194 (backport of carbon-design-system/carbon#2955)
This PR passes the
description
prop to the<Loading>
component SVG titleTesting / Reviewing
Ensure the SVG title reflects the
iconDescription
prop in<InlineLoading>
and thedescription
prop in<Loading>