bem / bem/bem-components

Review checkboxes and radio attr/prop usage

Open
#1,886 2 comments 0 reactions 0 assignees View on GitHub
a11y bug JS
Dominant language
JavaScript
Stars
330
Forks
88
PR merge metrics
No merged PRs in 30d

Description

It turned out that `form.reset()` works wrong with [current way](https://github.com/bem/bem-components/blob/v3/common.blocks/checkbox/checkbox.js#L17-L19) of checking/unchecking checkboxes.

Such variant was introduced in https://github.com/bem/bem-components/commit/012de48146234dc5c1ea47297687046311a0ac6b as a fix for previous change made for a11y: https://github.com/bem/bem-components/commit/db4a1385029ee328550aaacb541b41e021381fea

Is there any way to satisfy a11y and get rid of `attr()` setter?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.