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

Make the error link only apply to the first label #43

Merged
merged 1 commit into from
Sep 28, 2019

Conversation

peteryates
Copy link
Member

Previously it was affecting all labels in the collection and causing
invalid markup. Now the first label is associated with the first
input and when errors are present on the attribute both id and for are
changed accordingly

Previously it was affecting all labels in the collection and causing
invalid markup. Now the first label is associated with the first
input and when errors are present on the attribute both id and for are
changed accordingly
@peteryates peteryates added the bug Something isn't working label Sep 28, 2019
@peteryates peteryates merged commit b7f245f into master Sep 28, 2019
peteryates added a commit that referenced this pull request Oct 1, 2019
Similar to #43, this fixes a bug with fieldsets with errors where all
labels were associated with the first input.
@peteryates peteryates deleted the fix-radio-button-error-target branch October 18, 2019 10:12
peteryates added a commit that referenced this pull request Jan 9, 2020
Make the error link only apply to the first label.

Previously it was affecting all labels in the collection and causing
invalid markup. Now the first label is associated with the first
input and when errors are present on the attribute both id and for are
changed accordingly
peteryates added a commit that referenced this pull request Jan 9, 2020
Similar to #43, this fixes a bug with fieldsets with errors where all
labels were associated with the first input.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant