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(code-snippet): remove min-width rules for improved responsiveness #7420

Merged

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Dec 8, 2020

Closes #7406

This PR removes the min-width rule on single and multiline code snippets to avoid overflow at smaller screen sizes

Changelog

Changed

  • multiline CodeSnippet story styles

Removed

  • single and multiline CodeSnippet min-width rules

Testing / Reviewing

View the single and multiline code snippet stories at small screen sizes (<320 px wide) and confirm that the component appears properly

@netlify
Copy link

netlify bot commented Dec 8, 2020

✔️ Deploy preview for carbon-elements ready!

🔨 Explore the source changes: 3d9f9a6

🔍 Inspect the deploy logs: https://app.netlify.com/sites/carbon-elements/deploys/5fd14418f4e18800073f3f30

😎 Browse the preview: https://deploy-preview-7420--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Dec 8, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 3d9f9a6

https://deploy-preview-7420--carbon-components-react.netlify.app

Copy link
Member

@laurenmrice laurenmrice left a comment

Choose a reason for hiding this comment

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

Looks good to me ! 👍🏻

@kodiakhq kodiakhq bot merged commit 09c3bfe into carbon-design-system:master Dec 9, 2020
@emyarod emyarod mentioned this pull request Dec 10, 2020
39 tasks
@emyarod emyarod deleted the 7406-code-snippet-min-width branch December 10, 2020 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[code-snippet]: min-width causes reflow (400%) violation
4 participants