docs: update modal images with border on various component pages part 1#4781
Conversation
There was a problem hiding this comment.
Thank you @laurenmrice to work on this!
I have noticed some items to consider looking into:
Button:
Usage tab:
- Under the "Alignment" section in the images the side panel does not seem to have a border
- Under the "Horizontally arranged groups" section in the images of the side panel and modal does not seem to have a border
- Under the "Universal behaviors" section in the images the modal the border seems more white
- Under the "Loading" section in the images the modal animation does not seem to have a border
- Under the "Tertiary buttons in button groups " section in the images the side panel does not seem to have a border
- Under the "Ghost button used as a cancel action" section in the images the tearsheet does not seem to have a border (Modal, Side panel, and Tearsheet should all have a border)
Style tab:
- Under the "Button groups structure" section in the images of the side panel(s) does not seem to have a border
|
@sunny-babbar Just to address some of your comments below: The images for these are actually correct, I rechecked them. You can see them in the Usage and Style Figma file. Side panels Modals Gifs Tearsheet |
There was a problem hiding this comment.
Additionally here are some more comments:
@laurenmrice
I have noticed some items to consider looking into:
Inline loading:
Usage tab:
- Under the "Interactions" section in the images the modal does not seem to have a border
@laurenmrice Thank for your comments on this to help walkthrough my comments. As far as the seperate issues for Gifs and Tearsheet that is great that will be addressed. Those were my comments other than that everything else is good to go. |
sunny-babbar
left a comment
There was a problem hiding this comment.
Based off our conversation and comments this is looking good to me.




Closes #4766
This PR updates any modal images with the new a11y border across several component pages. This addresses half of our component pages and a separate PR will update the others. Side panels in images have also been updated with the border if it did not include it.
Changed
Updated current modal images with a11y border.