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

fix(content-group-cards): fixing extra overflow in medium resolution #12156

Conversation

bruno-amorim
Copy link
Contributor

Related Ticket(s)

Closes #
https://jsw.ibm.com/browse/ADCMS-7030

Description

In medium resolution, the cards are having an undesired horizontal overflow that was causing a scroll bar to appear.
I made a fix that will make them span full width, that's the behavior we have in the v2 builds I found for example, that came from this issue

Fixed as per this PR:
image

Changelog

New

setting margin-inline-start to be 0 in resolutions lower than LG.

@bruno-amorim bruno-amorim requested a review from a team as a code owner December 10, 2024 19:57
@bruno-amorim bruno-amorim requested review from m4olivei, marcelojcs, andy-blum and Valentin-Sorin-Nicolae and removed request for a team December 10, 2024 19:57
Copy link

netlify bot commented Dec 10, 2024

Deploy Preview for ibm-dotcom-web-components-react-wrap ready!

Name Link
🔨 Latest commit 38dd2cd
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components-react-wrap/deploys/67589d1e13997700084b258e
😎 Deploy Preview https://deploy-preview-12156--ibm-dotcom-web-components-react-wrap.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 10, 2024

Deploy Preview for ibm-dotcom-web-components ready!

Name Link
🔨 Latest commit adc2200
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components/deploys/6758a7468ef1b20009eb5a3a
😎 Deploy Preview https://deploy-preview-12156--ibm-dotcom-web-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andy-blum andy-blum changed the base branch from main to release/v2.16.0 December 10, 2024 20:40
@andy-blum andy-blum changed the base branch from release/v2.16.0 to main December 10, 2024 20:41
@andy-blum andy-blum closed this Dec 10, 2024
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.

2 participants