ENG-3764 A11y - OSF Home/Landing Page

[uditijmehta]

• Ticket: ENG-3764
• Feature flag: n/a

Purpose

Fix a11y issue for carousel by removing nested interactive controls

Summary of Changes

• Replaced 'li' elements with 'div' for carousel navigation
• Adjusted CSS to apply styles to the new 'div' structure
• Updated TypeScript logic to manage active state

Screenshot(s)

QA Notes

[brianjgeiger]

Overall, looks good, but you've broken a test. Integration | Component | carousel: li navigation works

[uditijmehta]

@brianjgeiger Fixed the failing test to match the updated div-based navigation. Carousel should be all set now.

[coveralls]

Pull Request Test Coverage Report for Build 8512978031

Details

• 8 of 8 (100.0%) changed or added relevant lines in 1 file are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.03%) to 68.147%

---

Files with Coverage Reduction	New Missed Lines	%
lib/osf-components/addon/components/carousel/x-item/component.ts	1	72.22%

Totals
Change from base Build 8345343041:	-0.03%
Covered Lines:	6139
Relevant Lines:	8760

---

💛 - Coveralls

[futa-ikeda]

Good stuff!