While the form elements function with or without the calendar and/or javascript, including the calendar icon in the form adds no value to screen readers since you can only navigate into that element. You can't actually use it. Your only option is to tab through it.

The calendar icon should be hidden from screen readers using aria. Patch to follow.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kreynen’s picture

DanChadwick’s picture

Category: Bug report » Feature request
Status: Active » Needs review

Seems reasonable. I'd like someone else with accessibility experience to review this and mark it RTBC.

DanChadwick’s picture

Issue tags: +Accessibility, +aria
Liam Morland’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

DanChadwick’s picture

Version: 7.x-4.x-dev » 8.x-4.x-dev
Status: Reviewed & tested by the community » Postponed

All aria features are being deferred until D8. Marking as postponed so this can be tended to after the port is functional.

fenstrat’s picture

Status: Postponed » Closed (outdated)

Closing to clear out the old Webform 8.x-4.x branch. See #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x.

kreynen’s picture

Version: 8.x-4.x-dev » 7.x-4.x-dev
Status: Closed (outdated) » Reviewed & tested by the community

Reopening. This was RBTC for D7. Now that it's clear nothing from D8 will be backported to D7, can we just apply the patch that was RTBC for D7.

  • Liam Morland committed d943e7e on 7.x-4.x authored by kreynen
    Issue #2534098 by kreynen: Add aria-hidden="true" and role="presentation...
Liam Morland’s picture

Status: Reviewed & tested by the community » Fixed

  • Liam Morland committed d943e7e on 8.x-5.x authored by kreynen
    Issue #2534098 by kreynen: Add aria-hidden="true" and role="presentation...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.