Skip to content

Conversation

@kyledurand
Copy link
Member

@kyledurand kyledurand commented Nov 4, 2021

WHY are these changes introduced?

Fixes #4560

Reverts the change added here: https://github.com/Shopify/polaris-react/pull/4556/files#diff-12888031b1f0e16bf9d7ee02978c37cbdfd6b64f8912ad164ab1cc51410fd538R10

We've decided to ditch this effort for now and rely on consumers to render Stack.Items only if their content is not null in the same way we do with Cards and Layout.Sections

@kyledurand kyledurand requested a review from chloerice November 4, 2021 14:52
@kyledurand kyledurand self-assigned this Nov 4, 2021
@kyledurand kyledurand force-pushed the stack_revert-not-empty branch from ae166ee to 2b27167 Compare November 4, 2021 14:52
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2021

size-limit report

Path Size
cjs 165.87 KB (0%)
esm 96.33 KB (0%)
esnext 143.22 KB (-0.01% 🔽)
css 34.52 KB (-0.03% 🔽)

Copy link
Member

@chloerice chloerice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@kyledurand kyledurand merged commit 1e6b37e into main Nov 4, 2021
@kyledurand kyledurand deleted the stack_revert-not-empty branch November 4, 2021 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Stack] Vertical alignment is off

2 participants