Skip to content
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

Regression - alignment on icons in Discount Badges #12468

Open
iphipps opened this issue Jul 26, 2024 · 2 comments
Open

Regression - alignment on icons in Discount Badges #12468

iphipps opened this issue Jul 26, 2024 · 2 comments
Labels
Bug Something is broken and not working as intended in the system. untriaged

Comments

@iphipps
Copy link

iphipps commented Jul 26, 2024

Summary

In web admin at the discounts creation modal, the icons are misaligned

Screenshot 2024-07-26 at 9 40 28 AM

link to the react component

Expected behavior

Aligned icons

Actual behavior

misaligned icons

Steps to reproduce

Link to sandbox

  1. spin url or go to https://admin.shopify.com/store/{your store}/discounts
  2. click the create discount button to see the icons

Are you using React components?

Yes

Polaris version number

"@shopify/polaris-icons": "9.3.0",     "@shopify/polaris-internal": "13.21.1",     "@shopify/polaris-tokens": "9.3.0",     "@shopify/polaris-viz": "14.2.0",     "@shopify/polaris-viz-core": "14.2.0",

Browser

chrome

Device

mac os

@iphipps iphipps added Bug Something is broken and not working as intended in the system. untriaged labels Jul 26, 2024
@sgilroy85
Copy link

This is also affecting the Shipping Label bulk purchase page. The order cells with the globe icon and the shipping service cells with the info icon are now showing a scroll bar. If we delete the icon from the dom the scrollbar disappears.
image

@willnguyen1312
Copy link
Member

Hi guys, I think this is the culprit causing our layout in ShippingLabels section to be broken related to Icon's styling

https://github.com/Shopify/polaris-internal/commit/b0812db3e5f256cd47645053385673d6c697e8a3#diff-ca33e9d43e366c169d1bcc09b4fb1053f1aa766f958aacf7578bc9ed90bf262dL94

Previously, Icon's svg was using display: block instead of its default inline. The latest style doesn't work nicely 😅

Weird.CSS.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken and not working as intended in the system. untriaged
Projects
None yet
Development

No branches or pull requests

3 participants