Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Checkbox v2 Component #4471

Closed
traviskaufman opened this issue Jun 29, 2016 · 2 comments
Closed

Implement Checkbox v2 Component #4471

traviskaufman opened this issue Jun 29, 2016 · 2 comments

Comments

@traviskaufman
Copy link
Contributor

Spec reference: https://www.google.com/design/spec/components/selection-controls.html#selection-controls-checkbox

@ianspiro
Copy link

ianspiro commented Aug 4, 2016

There's a small accessibility bug with the current checkbox. Screenreaders (ChromeVox, at least) use the underlying checkbox input for highlighting. The original input is offset somewhat from the upgraded version. Here's a screenshot. Could you keep this in mind with the v2 component? Thanks!

checkbox

@Garbee
Copy link
Collaborator

Garbee commented Aug 5, 2016

Yup, Travis is working on this honking component currently. Once it is up for review I'll be running it through Chromevox and NVDA to test. This is great feedback to keep an eye on. Thank you.

traviskaufman added a commit that referenced this issue Aug 5, 2016
This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
traviskaufman added a commit that referenced this issue Aug 5, 2016
This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
traviskaufman added a commit that referenced this issue Aug 8, 2016
This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
traviskaufman added a commit that referenced this issue Aug 8, 2016
This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
traviskaufman added a commit that referenced this issue Aug 8, 2016
This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
traviskaufman added a commit that referenced this issue Aug 8, 2016
* feat(animation): Clean up mdl-animation, add README

* feat(checkbox): Add initial checkbox implementation

This commit converts the checkbox POC code over to the new architecture, adds unit tests, and ports
the demos over to working implementations. It also adds a README, and
modifies .stylelintrc.yaml to increase the number of compound selectors
we can use.

Note that ink ripple functionality will be covered in a separate issue.

Finishes #4471

[Delivers #126819487]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants