Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Replaces usage of SelectDropdown in ActionSelect with DropList #933

Merged
merged 4 commits into from Jun 4, 2021

Conversation

tinkertrain
Copy link
Collaborator

The Dropdown obliteration continues with the last component using it inside HSDS: ActionSelect.

Straightforward refactoring that doesn't change the original interface of ActionSelect to avoid having to change any usages inside of HSApp or else.

馃摴 Original

馃摴 Refactored

@tinkertrain tinkertrain requested a review from plbabin May 26, 2021 21:06
@tinkertrain tinkertrain self-assigned this May 26, 2021
@tinkertrain tinkertrain changed the title Replaces usage of SelectDropdown with DropList Replaces usage of SelectDropdown in ActionSelect with DropList May 26, 2021
@netlify
Copy link

netlify bot commented May 26, 2021

鉁旓笍 Deploy Preview for hsds-react ready!

馃敤 Explore the source changes: 1c3d4a8

馃攳 Inspect the deploy log: https://app.netlify.com/sites/hsds-react/deploys/60ba47706158190007faadca

馃槑 Browse the preview: https://deploy-preview-933--hsds-react.netlify.app/iframe

@coveralls
Copy link

coveralls commented May 26, 2021

Pull Request Test Coverage Report for Build 3608

  • 14 of 18 (77.78%) changed or added relevant lines in 1 file are covered.
  • 12 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.2%) to 93.767%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/ActionSelect/ActionSelect.jsx 14 18 77.78%
Files with Coverage Reduction New Missed Lines %
src/components/Dropdown/Dropdown.actions.js 1 59.45%
src/components/Dropdown/Dropdown.Card.jsx 3 64.29%
src/components/ActionSelect/ActionSelect.jsx 4 54.22%
src/components/SelectDropdown/SelectDropdown.jsx 4 89.55%
Totals Coverage Status
Change from base Build 3607: -0.2%
Covered Lines: 6584
Relevant Lines: 6846

馃挍 - Coveralls

Copy link
Collaborator

@plbabin plbabin left a comment

Choose a reason for hiding this comment

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

amazing work 馃帀 馃帀 馃帀

@tinkertrain tinkertrain merged commit 1c3d4a8 into master Jun 4, 2021
@tinkertrain tinkertrain deleted the refactor/actionselect branch June 4, 2021 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants