Problem/Motivation

If you include a video asset using EMBED CODE formatter as a background / decorative element it will trigger warnings in WCAG and ADA checks due to missing information, i.e.:

Failed Checks:

  • Video element visual-only content has accessible alternative.
  • Video element auditory content has accessible alternative
  • Video element visual content has accessible alternative

Steps to reproduce

  • Add a video in the DAM.
  • Add content type with a media field that allows Acquia DAM: Video
  • Configure Acquia DAM: Video formatter to Browser HTML5 player with loop, autoplay, muted
  • Create a new page, select the video, publish page
  • Run accessibility scans on page

    • Proposed resolution

    Provide a "decorative" option in the embed code formatter settings which, when toggled on, will add aria-hidden="true" to the embed.

    Remaining tasks

    1. Add decorative option to embed code formatter settings
    2. Adjust EmbedCodeFormatter to add the aria-hidden="true" attribute when the setting is TRUE/1
    3. Maybe write a test for it 😀

    User interface changes

    New toggle on the field display settings for embed code formatter

    API changes

    Data model changes

    Issue fork acquia_dam-3542480

    Command icon Show commands

    Start within a Git clone of the project using the version control instructions.

    Or, if you do not have SSH keys set up on git.drupalcode.org:

    About issue forks

    Comments

    dan612 created an issue. See original summary.

    dan612 opened merge request !189

    dan612’s picture

    dan612 commented
    Issue summary: View changes
    dan612’s picture

    dan612 commented

    test could probably be included in EmbedCodeFormatterTest

    rajeshreeputra made their first commit to this issue’s fork.

    rajeshreeputra opened merge request !282

    rajeshreeputra’s picture

    rajeshreeputra
    he/him
    Location Pune
    commented
    Status: Active » Needs work

    Code coverage will be added soon.

    • rajeshreeputra committed 97a3ec00 on 1.1.x 
      Resolve #3542480 "Provide decorative image option for Acquia DAM Image...
    rohan-sinha’s picture

    rohan-sinha commented
    Status: Needs work » Reviewed & tested by the community

    lgtm.

    rajeshreeputra’s picture

    rajeshreeputra
    he/him
    Location Pune
    commented
    Status: Reviewed & tested by the community » Fixed

    MR merged!

    Now that this issue is closed, review the contribution record.

    As a contributor, attribute any organization that helped you, or if you volunteered your own time.

    Maintainers, credit people who helped resolve this issue.

    Status: Fixed » Closed (fixed)

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