-
Notifications
You must be signed in to change notification settings - Fork 141
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
USWDS-Site: Add changelogs for autocomplete email attribute [#6002] #2765
base: release-3.9.0
Are you sure you want to change the base?
Conversation
todo: This feels like a good one to make a note of in the release notes so users know they can improve their implementation by adding this attribute. |
test_status: exception | ||
test_status: pass | ||
test_type: general | ||
status_details: The input is appropriately marked as an "email" input type, but does not have the autocomplete attribute in the default component example. Be sure to add the `autocomplete="email"` attribute to the email input element in your implementation of the component. | ||
github_issue_number: 6001 | ||
github_issue_repo: uswds | ||
version_tested: 3.8.1 | ||
version_tested: N.N.N |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note
PR uswds/uswds#6002 resolves an accessibility issue that was previously marked as "passed with exceptions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, test_status: pass
github_issue_number: 6001 | ||
github_issue_repo: uswds | ||
version_tested: 3.8.1 | ||
version_tested: N.N.N |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note
We should update this value to match its related release.
version_tested: N.N.N | |
version_tested: 3.9.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM too
Summary
Important
We need to update the changelog dates before merge.
Related issue
Related to uswds/uswds#6001
Preview link
Testing and review