ni / nimble

The NI Nimble Design System
https://nimble.ni.dev
MIT License
32 stars 8 forks source link

Checkbox error state HLD update #2467

Closed mollykreis closed 1 week ago

mollykreis commented 1 week ago

Pull Request

๐Ÿคจ Rationale

During development of #2018, I realized that implementing the error state of the checkbox the way it was spec'ed out would be much more difficult than originally thought because making only the label and square checked indicator clickable would break a number of tests (inside & outside of nimble) and would be against ARIA guidelines. Therefore, we decided to keep the effort of the feature more in line with what our original estimates were by deferring the work to update which portion of the control was clickable.

๐Ÿ‘ฉโ€๐Ÿ’ป Implementation

Update HLD

๐Ÿงช Testing

None

โœ… Checklist

mollykreis commented 1 week ago

@m-akinc, will you buddy this PR for me?

jattasNI commented 1 week ago

This is more of a UX / scoping decision than a technical one so I added @fredvisser as a reviewer and I think it's ok not to wait for @rajsite. I'm also ok with verbal signoff from Fred.

I chatted with @fredvisser offline and he said he is ok with this change.