-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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
[FIX] Retry icon comes out of the div #20390
Conversation
Instead of changing the sizes, wouldn't a simple padding in the placeholder suffice? |
Please, also post screenshots of the fix. Thanks! |
Sure! This seems to be better indeed!! |
This pull request introduces 16 alerts when merging fa495c6 into 54e0a71 - view on LGTM.com new alerts:
|
@dougfabris @ggazzo can anyone please review this? 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.
Other suggestion, remove the {...props}
in that component, there is no necessity to pass dimensions for that.
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.
The { ...dimensions }
on Retry componente must be removed... anyways, approved.
Thank you @tiagoevanp! Removed {...dimensions} as well! |
This pull request introduces 1 alert when merging 4728cc0 into 4f05c55 - view on LGTM.com new alerts:
|
Proposed changes (including videos or screenshots)
Changed the height of the div container.
Issue(s)
This PR closes #20389
Earlier
Now
Note
Due to different image sizes, the widths are different here in the image but actually the boxes are of the same width and the retry icon and text are of the same size.