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

style(leadspace): added explicit height declaration for sections #4763

Merged

Conversation

guilhermelMoraes
Copy link
Contributor

Description

After the ::part pseudo-element introduction (#4752) to change the leadspace height according to the Partner world landing page requirements, I've missed adding an explicit height: 100% declaration for the wrappers below it. This is causing the leadspace to be visually broken for the md and sm breakpoints.

@ibmdotcom-bot
Copy link
Contributor

@ibmdotcom-bot
Copy link
Contributor

@ibmdotcom-bot
Copy link
Contributor

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @guilhermelMoraes!

@asudoh asudoh added package: react Work necessary for the Carbon for IBM.com react components package package: web components Work necessary for the IBM.com Library web components package labels Dec 22, 2020
@guilhermelMoraes guilhermelMoraes added the Ready to merge Label for the pull requests that are ready to merge label Dec 22, 2020
@kodiakhq kodiakhq bot merged commit 91c3c17 into carbon-design-system:master Dec 22, 2020
@guilhermelMoraes guilhermelMoraes deleted the fix/leadspace-height branch December 22, 2020 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: react Work necessary for the Carbon for IBM.com react components package package: web components Work necessary for the IBM.com Library web components package Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants