Skip to content

Conversation

@cg-julian-taylor
Copy link
Collaborator

Separate radio groups and checkbox groups into their own components.

cg-julian-taylor and others added 8 commits December 14, 2022 16:58
* feat: Deprecate <FormRadio>

* feat: add <FormRadio /> component

* feat: add <FormRadioCheckboxBase> component

* feat: update dev script

* feat: move <FormGroup /> to <FormRadio />

* feat: rename test FormGroup to RadioGroup

* feat: add storybook documentation

* feat: Rename StyledRadios to Styled

* feat: Fix unstyled stories
* feat: create <CheckboxGroup /> component

* feat: update test

* feat: fix test. Was looking for wrong element
@cg-julian-taylor cg-julian-taylor marked this pull request as ready for review December 20, 2022 12:11
@codecov-commenter
Copy link

codecov-commenter commented Dec 20, 2022

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage 👍

Coverage data is based on head (479da57) compared to base (d3f69b1).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##           release/0.6      #380   +/-   ##
=============================================
  Coverage       100.00%   100.00%           
=============================================
  Files               42        44    +2     
  Lines              998      1005    +7     
  Branches           390       391    +1     
=============================================
+ Hits               998      1005    +7     
Impacted Files Coverage Δ
src/formRadio/FormRadio.tsx 100.00% <ø> (ø)
src/checkboxGroup/CheckboxGroup.tsx 100.00% <100.00%> (ø)
src/common/components/FormRadioCheckboxBase.tsx 100.00% <100.00%> (ø)
src/radioGroup/RadioGroup.tsx 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@daniele-zurico daniele-zurico merged commit e29e11b into release/0.6 Dec 20, 2022
@daniele-zurico daniele-zurico deleted the feature/radio-and-checkbox branch December 20, 2022 16:30
daniele-zurico added a commit that referenced this pull request Jan 18, 2023
* feat: add value to button

* feat:select-containerId (#350)

* fix: On initial render, ensure that the input prompt for Autocomplete is not visible unless focused (#357)

* Feature/upgrade deps (#359)

* feat: upgrade deps

* feat: downgrade microbundle

* Autocomplete: Ensure the value of controlled input is never undefined (#358)

* feat: upgrade microbundle (#360)

* fix: Accessibility improvements (#361)

* Feature/autocomplete (#364)

* feat: add dynamic id to the list

* feat: add dynamic id to the list

* feat: change on the id

* feat: remove jss dependecy (#365)

* Make onChange prop optional for FormGroup component (#368)

* feat: use sass library instead of node-sass

* feat: onChange type now optional

* Separate radio and checkbox groups (#380)

* feat: remove <FormRadio> storybook (#369)

* feat: Deprecate <FormRadio> (#370)

* feat: remove <FormGroup /> and related stories

* Task create form radio (#371)

* feat: Deprecate <FormRadio>

* feat: add <FormRadio /> component

* feat: add <FormRadioCheckboxBase> component

* feat: update dev script

* feat: move <FormGroup /> to <FormRadio />

* feat: rename test FormGroup to RadioGroup

* feat: add storybook documentation

* feat: Rename StyledRadios to Styled

* feat: Fix unstyled stories

* Task create checkbox group (#373)

* feat: create <CheckboxGroup /> component

* feat: update test

* feat: fix test. Was looking for wrong element

* feat: update storybook documentation (#378)

* feat: remove "unstyled" dir from radioGroup story

* feat: update storybook doc for single checkbox (#381)

* Correct single checkbox stories (#382)

* feat: correct <FormCheckbox /> Stories

* feat: correct <FormCheckbox /> Stories

* feat: add class to fix styling (#383)

* fix: add correct components to example src (#385)

* feat: move elements outside of label (#387)

* Conditionally render aria-label (#386)

* feat: conditionally render aria-label

* feat: unit test failed eslint checks

* Feature/upgrade storybook (#392)

* feat: update storybook

* feat: update storybook

* Feature/tab index (#389)

* fix: tabIndex prop for Button CopyToClipboard Details and FormDate

* fix: tabIndex type to string

* fix: tabIndex back to number

* fix: tabIndex and missing tests

* fix: updated liveEdit files

Co-authored-by: Daniele Zurico <daniele.zurico@gmail.com>

* fix: allow closing of the options list with the Escape key instead of onBlur (#400)

* Feat/update readme (#391)

* Update CHANGELOG.md

Co-authored-by: SergioBandera <113599747+SergioBanderaGarcia@users.noreply.github.com>
Co-authored-by: Sri Viswanathan <sri@protodiv.com>
Co-authored-by: cg-julian-taylor <107050226+cg-julian-taylor@users.noreply.github.com>
Co-authored-by: Szilvi Horvath <szilvi.horvath@capgemini.com>
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.

4 participants