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): set correct width for feedback container in Safari #5601

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Mar 11, 2020

Closes #5595

This PR resets the copy button feedback container box-sizing so that max-content works correctly in Safari. Not sure if this would be better in a Safari specific media query but the other browsers appear unaffected

Testing / Reviewing

Ensure that the copy button feedback has the correct width in code snippets and in the individual copy button component

@netlify
Copy link

netlify bot commented Mar 11, 2020

Deploy preview for carbon-components-react ready!

Built with commit ea6540e

https://deploy-preview-5601--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Mar 11, 2020

Deploy preview for carbon-elements ready!

Built with commit ea6540e

https://deploy-preview-5601--carbon-elements.netlify.com

Copy link
Contributor

@abbeyhrt abbeyhrt 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! Checked Edge, FF, Safari, and Chrome on mac and all seems good!

@asudoh asudoh requested review from a team and laurenmrice and removed request for a team March 11, 2020 22:48
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 !

@asudoh
Copy link
Contributor

asudoh commented Mar 12, 2020

@emyarod thinks this is ready for merge - Thanks @emyarod!

@asudoh asudoh merged commit 778c622 into carbon-design-system:master Mar 12, 2020
@emyarod emyarod deleted the 5595-safari-copy-button-feedback-width branch March 13, 2020 00:13
This was referenced Mar 18, 2020
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
renmaddox added a commit to renmaddox/carbon that referenced this pull request Mar 20, 2020
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 tooltip cut off on Safari
5 participants