Skip to content

fix(radio-button): Squished on overflow text#323

Merged
tw15egan merged 1 commit into
carbon-design-system:masterfrom
jamesvclements:fix-squished-radio-button
Sep 21, 2017
Merged

fix(radio-button): Squished on overflow text#323
tw15egan merged 1 commit into
carbon-design-system:masterfrom
jamesvclements:fix-squished-radio-button

Conversation

@jamesvclements

Copy link
Copy Markdown
Contributor

Overview

Resolves #322

Adds flex shrink to prevent distorting radio button. Check it out here by removing the commented out style.

@marijohannessen

Copy link
Copy Markdown
Contributor

👍 ✅ Looks good to me!

@tw15egan

Copy link
Copy Markdown
Contributor

👍 ✅ Thanks for fixing that!

@tw15egan tw15egan merged commit 16e6927 into carbon-design-system:master Sep 21, 2017
@jamesvclements jamesvclements deleted the fix-squished-radio-button branch September 21, 2017 21:52
joshblack pushed a commit to joshblack/carbon that referenced this pull request May 2, 2019
…#323)

componentWillReceiveProps was doing a check on object vs object. Even if
the objects had the same props, it would always return false because
JavaScript. Doing a very simple toString-like check is a better check

updatePosition changed too much state and fired onChange too much as
well. It would fire on every mouse move, even if the value didn't
actually change

Fixes carbon-design-system#329
joshblack pushed a commit to joshblack/carbon that referenced this pull request May 8, 2019
…n-system#323)

* fix $expressive-heading-04 line height at mobile

Line height should be the same at every breakpoint at Step 6 for $expressive-heading-04; it only changes when the type size changes at max breakpoint to step 7.

* fix(type): update line-height in scss

* chore(test): update snapshots
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.

3 participants