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

feat #1677 WAI-ARIA - DatePicker - Reading the input date when valid #1677

Merged
merged 2 commits into from
Sep 8, 2016

Conversation

ymeine
Copy link
Contributor

@ymeine ymeine commented Sep 7, 2016

Now when the user inputs a date, be it from the calendar or when typing in, if the date is valid it is read by the screen reader following a provided format: waiAriaConfirmDateFormat (which defaults to waiAriaDateFormat or a long date format).

When the user types the text in, a debouncing algorithm is applied, which is controlled by a delay waiAriaConfirmDateDelay, defaulting to 50 ms.

References:

  • AT-1509

if (onCall == null) {
onCall = nop;
}
this.onCall = onCall;
Copy link
Member

Choose a reason for hiding this comment

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

onCall is not used anywhere. Please remove it.

@ymeine ymeine force-pushed the feat/widgets/wai/date-picker branch from 1b7e9cd to 4540a17 Compare September 8, 2016 08:55
ymeine added a commit to ymeine/ariatemplates that referenced this pull request Sep 8, 2016
…e when valid

Now when the user inputs a date, be it from the calendar or when typing in, if the date is valid it is read by the screen reader following a provided format: waiAriaConfirmDateFormat (which defaults to waiAriaDateFormat or a long date format).

When the user types the text in, a debouncing algorithm is applied, which is controlled by a delay waiAriaConfirmDateDelay, defaulting to 50 ms.

References:

- AT-1509

close ariatemplates#1677
…nk widget

Also made the following change to all "ActionWidget" based widgets: now the "aria-disabled" attribute is not set anymore in WAI-ARIA mode, "disabled" must be sufficient.

References:

- AT-1398
- PTR 11630074

close ariatemplates#1675
@ymeine ymeine force-pushed the feat/widgets/wai/date-picker branch from 4540a17 to 4056491 Compare September 8, 2016 14:20
ymeine added a commit to ymeine/ariatemplates that referenced this pull request Sep 8, 2016
…e when valid

Now when the user inputs a date, be it from the calendar or when typing in, if the date is valid it is read by the screen reader following a provided format: waiAriaConfirmDateFormat (which defaults to waiAriaDateFormat or a long date format).

When the user types the text in, a debouncing algorithm is applied, which is controlled by a delay waiAriaConfirmDateDelay, defaulting to 50 ms.

References:

- AT-1509

close ariatemplates#1677
…e when valid

Now when the user inputs a date, be it from the calendar or when typing in, if the date is valid it is read by the screen reader following a provided format: waiAriaConfirmDateFormat (which defaults to waiAriaDateFormat or a long date format).

When the user types the text in, a debouncing algorithm is applied, which is controlled by a delay waiAriaConfirmDateDelay, defaulting to 50 ms.

References:

- AT-1509

close ariatemplates#1677
@ymeine ymeine force-pushed the feat/widgets/wai/date-picker branch from 4056491 to fe5721d Compare September 8, 2016 14:47
@ymeine ymeine merged commit fe5721d into ariatemplates:master Sep 8, 2016
divdavem pushed a commit that referenced this pull request Sep 16, 2016
Now when the user inputs a date, be it from the calendar or when typing in, if the date is valid it is read by the screen reader following a provided format: waiAriaConfirmDateFormat (which defaults to waiAriaDateFormat or a long date format).

When the user types the text in, a debouncing algorithm is applied, which is controlled by a delay waiAriaConfirmDateDelay, defaulting to 50 ms.

References:

- AT-1509

close #1677

(cherry picked from commit fe5721d)
@divdavem divdavem added this to the 1.7.23 milestone Oct 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants