SHS-6103: ARIA - "ARIA attribute unsupported or prohibited" in Vertical Timelines

[mariannuar]

Summary

Fix aria attributes in Vertical Timelines

Need Review By (Date)

03/12

Urgency

medium

Steps to Test

1. Go to https://pr1766-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com/components/vertical-timeline-without-and-collection
2. Test by using a screen reader and keyboard navigation, and confirm everything is working well.
3. Aria attributes are not necessary when using
   and

   in the markup

Review Tasks

Backend / Functional Validation

Code

• Are the naming conventions following our standards?
• Are PHP functions and variables in snake_case and not camelCase?
• Does Drupal code follow Drupal Coding Standards?
• Does the code have sufficient inline comments?
• Is there anything in this code that would be hidden or hard to discover through the UI?
• Are there any code smells?
• Are tests provided?

Code security

• Is all user input properly sanitized when rendered?
• Any obvious security flaws or new areas for attack?

General

• Is there anything included in this PR that is not related to the problem it is trying to solve?
• Is the approach to the problem appropriate?

[ahughes3]

Tugboat has finished building the preview for this pull request!

Links:

• https://pr1766-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com
• https://hs-colorful-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com
• https://hs-traditional-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com
• https://history-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com
• https://west-oul53tg46g49olxd2awym2ihyx59xlac.tugboatqa.com

Dashboard:

• https://dashboard.tugboatqa.com/67c5d341b18a3c1feb06a260

Visual Diffs:

• https://dashboard.tugboatqa.com/67c5d341b18a3c1feb06a260/visualdiffs

[cienvaras]

@mariannuar Looks good! Just a minor fix needed.

[cienvaras]

This comment is misleading, please remove it. The open property is not add via javascript, the browser adds it automatically.