-
Notifications
You must be signed in to change notification settings - Fork 178
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(app): fix the space between icon and banner outline #15082
Conversation
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.
If we are merging this into edge
, and we'll have the time to address broader DQA feedback before next release, do you think it's worthwhile just adding this offset directly to the banner component?
Just looking over all of its usages, I think this margin offset is probably safe to do everywhere. I imagine setting the icon/text a bit more left from the banner's edge will look fine, but if you or others disagree I'm fine with this change for now.
Actually, this PR should go to |
Think you may need to rebase the branch. I think it's bringing all the |
have failed rebase I will re-open a pr that is based on 7.3.0 since that will be faster than fixing rebase issue. |
Overview
fix the space between icon and banner outline
Actually, I don't like this fix personally since using margin isn't a good way to make a layout but Banner is used in many components, and we shouldn't touch the component itself since updating the component may cause other bugs.
We will need to update the component to align with the latest design.
design
https://www.figma.com/file/l7BAJAGICr10ELsSqQD0Sr/Design-System%3A-Desktop?node-id=7602-143972&mode=dev
[before]
[after]
fix RQA-2649
Test Plan
Changelog
Review requests
Risk assessment