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

Select component does not handle focus via label with labelId #6432

Closed
coreyworrell opened this issue Sep 18, 2024 · 0 comments
Closed

Select component does not handle focus via label with labelId #6432

coreyworrell opened this issue Sep 18, 2024 · 0 comments
Assignees
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Milestone

Comments

@coreyworrell
Copy link

coreyworrell commented Sep 18, 2024

const label = document.querySelector(`label[for="${this.inputId}"]`);

The docs show that inputId is deprecated and to use labelId instead. But the listener that is bound to the label only looks for inputId.

Binding to a single label element is not sufficient either, as multiple labels could be used for a single input (label above input, error message below input, label could be dynamically added, etc).

Another note: typically hovering a label also applies :hover to it's respective input, and this does not happen either.

@tugcekucukoglu tugcekucukoglu added this to the 4.1.0 milestone Oct 4, 2024
@tugcekucukoglu tugcekucukoglu added the Type: Bug Issue contains a bug related to a specific component. Something about the component is not working label Oct 4, 2024
@tugcekucukoglu tugcekucukoglu self-assigned this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Projects
None yet
Development

No branches or pull requests

2 participants