Skip to content
This repository has been archived by the owner on Oct 6, 2020. It is now read-only.

fix(Checkbox/Radio): Fixes regression from 5.1.1 update #99

Merged
merged 12 commits into from
Jan 29, 2020

Conversation

mathewmorris
Copy link
Contributor

@mathewmorris mathewmorris commented Jan 25, 2020

Coming from feedback in https://github.com/heydoctor/web-app/pull/408

This will solve the regressions from that update while still keeping the features requested.

@codecov
Copy link

codecov bot commented Jan 25, 2020

Codecov Report

Merging #99 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   66.15%   66.15%           
=======================================
  Files          21       21           
  Lines         458      458           
  Branches       93       93           
=======================================
  Hits          303      303           
  Misses        124      124           
  Partials       31       31
Impacted Files Coverage Δ
src/theme.js 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5596bb...e7449af. Read the comment docs.

@mathewmorris mathewmorris requested a review from a team January 25, 2020 01:03
@ghost ghost requested review from kylealwyn and schlegz January 25, 2020 01:04
@mathewmorris
Copy link
Contributor Author

@heydoctor/engineering Could I get a couple eyes on this when you get the chance?

src/Form/Checkbox.js Outdated Show resolved Hide resolved
src/Form/Checkbox.stories.js Show resolved Hide resolved
@alextranwork

This comment has been minimized.

Copy link
Contributor

@alextranwork alextranwork left a comment

Choose a reason for hiding this comment

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

one comment, otherwise lgtm

@mathewmorris
Copy link
Contributor Author

@erikshestopal Wanna give this another pass before merging?

Co-Authored-By: Erik Shestopal <[email protected]>
Copy link
Contributor

@erikshestopal erikshestopal left a comment

Choose a reason for hiding this comment

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

Looks great! Love the focus animations 😄 👍

@mathewmorris mathewmorris merged commit 2848510 into master Jan 29, 2020
@mathewmorris mathewmorris deleted the fix/checkbox-regression branch January 29, 2020 22:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants