Skip to content
This repository has been archived by the owner on Feb 29, 2020. It is now read-only.

Bug 1550642 - Improve accessibility of autofocused email #5016

Merged
merged 1 commit into from May 10, 2019

Conversation

rlr
Copy link
Contributor

@rlr rlr commented May 10, 2019

No description provided.

@rlr
Copy link
Contributor Author

rlr commented May 10, 2019

a11y panel picked it up:

Screen Shot 2019-05-10 at 8 24 21 AM

@rlr rlr changed the base branch from master to trailhead May 10, 2019 16:40
Copy link
Member

@Mardak Mardak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess not much we can do about the screen reader saying the email is "required" ?

@rlr
Copy link
Contributor Author

rlr commented May 10, 2019

hmm, looks like we can take out the required and it will still validate for a well formed email address. But if it's empty, it submits. So we could handle the one empty case separately in the onSubmit. Worth it?

@Mardak Mardak merged commit 98c25c4 into mozilla:trailhead May 10, 2019
@rlr rlr deleted the bug1550642/a11y branch May 14, 2019 20:17
Mardak pushed a commit to Mardak/activity-stream that referenced this pull request May 20, 2019
Mardak added a commit that referenced this pull request May 21, 2019
* chore(l10n): Update from l10n-central Mon May 20 2019 14:51:14 GMT-0700 (PDT)
* Bug 1550723 - An inactive horizontal scrollbar is displayed at the bottom of the about:welcome pages which contain a modal (#5017)
* Bug 1550566 - Retrieve `deviceID` parameter from FxA metrics endpoint and append as additional query parameter for FxA flows (#5015)
* Bug 1550642 - Improve accessibility of autofocused email (#5016)
* Bug 1550614 - Mobile image accidentally includes Lockwise in the background (#5019)
* Port 1550572 - More string updates for onboarding messaging r=flod,r1cky (#5021)
* Bug 1551207 - No telemetry ping is sent if the "First Run Cards" are dismissed (#5023)
* Bug 1551677 - Improve accessibility of trailhead dialog (#5028)
* Bug 1551832 - Onboarding button label not wrapping (#5032)
* Bug 1550649 - Should not roll trailhead branch if has addons attribution data (#5036)
* Bug 1550217 - Pref on Trailhead with desired experimentation for 67.0.5 (#5045)
* Bug 1550858 - Update final final final final final actions or urls (#5046)
* Bug 1553205 - FxA metrics values sometimes don't get set (#5053)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants