Skip to content

Commit 3f9456f

Browse files
PKulkoRaccoonGangvladislavkeblysh
authored andcommitted
feat: [AsPu-562] accessibility HTML attributes for Course Navigation top bar
1 parent 3cbbb02 commit 3f9456f

File tree

3 files changed

+10
-1
lines changed

3 files changed

+10
-1
lines changed

src/courseware/course/sequence/sequence-navigation/SequenceNavigation.jsx

+6-1
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,12 @@ const SequenceNavigation = ({
102102
};
103103

104104
return sequenceStatus === LOADED && (
105-
<nav id="courseware-sequence-navigation" data-testid="courseware-sequence-navigation" className={classNames('sequence-navigation', className, { 'mr-2': shouldDisplayNotificationTriggerInSequence })}>
105+
<nav
106+
id="courseware-sequenceNavigation"
107+
className={classNames('sequence-navigation', className)}
108+
style={{ width: shouldDisplayNotificationTriggerInSequence ? '90%' : null }}
109+
aria-label="course sequence tabs"
110+
>
106111
{renderPreviousButton()}
107112
{renderUnitButtons()}
108113
{renderNextButton()}

src/courseware/course/sequence/sequence-navigation/SequenceNavigationTabs.jsx

+1
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ const SequenceNavigationTabs = ({
3838
<div
3939
className="sequence-navigation-tabs d-flex flex-grow-1"
4040
style={shouldDisplayDropdown ? invisibleStyle : null}
41+
role="tablist"
4142
ref={containerRef}
4243
>
4344
{unitIds.map(buttonUnitId => (

src/courseware/course/sequence/sequence-navigation/UnitButton.jsx

+3
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ const UnitButton = ({
3939
variant="link"
4040
onClick={handleClick}
4141
title={title}
42+
role="tab"
43+
aria-selected={isActive}
44+
aria-controls={title}
4245
as={Link}
4346
to={unitPath}
4447
>

0 commit comments

Comments
 (0)