-
Notifications
You must be signed in to change notification settings - Fork 54
Reduce height of Brand and Script Link components #2569
Conversation
…ange-brand-height
@DenisHdz I just followed the same logic in my PR. |
Co-Authored-By: Denis Hernandez <46446236+DenisHdz@users.noreply.github.com>
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 👍
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.
👍 Looks good. We should also get this UX reviewed prior to merging.
This PR has already been reviewed and approved by UX |
Looks good to me. |
Resolves #2555
Overall change:
Changed the height of Script Link component for breakpoints above 600px and the height of Brand Component for all breakpoints.
Code changes: