Skip to content

Conversation

@ryanseddon
Copy link
Contributor

  • BREAKING CHANGE?

Description

Clicking on the open isolated button only for the checkboxes examples throws, see gif.

checkbox-throw

Detail

Seems for whatever reason that the name Checkbox somehow conflicts with react-styleguidist internals, they do have a Checkbox component that I tried renaming to see of that would fix the conflict but it still threw.

This is a quick fix.

Checklist

  • 👌 design updates are Garden Designer approved (add the
    designer as a reviewer)
  • 💅 view component styling is based on a Garden CSS
    component
  • 🌐 Styleguidist demo is up-to-date (yarn start)
  • ⬅️ renders as expected with reversed (RTL) direction
  • analyzed via axe and evaluated using VoiceOver
  • 💂‍♂️ includes new unit tests
  • 📝 tested in Chrome, Firefox, Safari, Edge, and IE11

@ryanseddon ryanseddon merged commit 12479b5 into master Aug 21, 2019
@ryanseddon ryanseddon deleted the ryan/styleguidist_throwing_bug branch August 21, 2019 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants