a11y issue: timeline is UL, should be OL.

Open imbrianj opened this issue 10 months ago • 1 comments

Description

The timeline component uses a UL for containing the timeline events: https://github.com/saas-js/saas-ui/blob/main/packages/saas-ui-core/src/timeline/timeline.tsx#L17

By definition, the timeline is an ordered list of chronological events. For a11y and general semantics, a OL is likely preferred here.

Link to Reproduction

No response

Steps to reproduce

Visit the Timeline component Inspect the Timeline events They are within an unordered list while this component is intended for chronologically ordered data (ordered list / OL).

Saas UI Version

2.11.4

Chakra UI Version

No response

Browser

No response

Operating System

[ ] macOS
[ ] Windows
[ ] Linux

Additional Information

No response

Feb 27 '25 23:02 imbrianj

SUI-628 a11y issue: timeline is UL, should be OL.

Feb 27 '25 23:02 linear[bot]