Skip to content

Conversation

@austingreendev
Copy link
Contributor

@austingreendev austingreendev commented Sep 3, 2019

Description

This PR introduces a Multiselect component within the react-dropdowns package. This is the final item necessary to finalize the react-autocomplete deprecation.

Detail

This component has a similar API to the previous element. It also includes the complications of that design so this isn't the most elegant solution. This component is starting to stretch the "global" <Dropdown> API that is shared with Menu, Select, and Autocomplete.

Downshift is working on a collection of hooks that would replicate what we have achieved in react-dropdowns. Hopefully we can transition to those once they are fleshed out some more.

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

@coveralls
Copy link

coveralls commented Sep 3, 2019

Coverage Status

Coverage increased (+0.07%) to 95.873% when pulling 502c5a4 on agreen/multiselect into 40fc779 on master.

@austingreendev austingreendev marked this pull request as ready for review September 10, 2019 18:10
@austingreendev
Copy link
Contributor Author

Coveralls now has a corrected coverage https://coveralls.io/builds/25638668

`}
`;

export const StyledMoreAnchor = styled.div`
Copy link
Member

Choose a reason for hiding this comment

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

Sadness that we can't modify our StyledAnchor here. 🙁

@austingreendev austingreendev merged commit a414ef9 into master Sep 18, 2019
@austingreendev austingreendev deleted the agreen/multiselect branch September 18, 2019 18:23
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.

5 participants