Skip to content

Commit a05d024

Browse files
Merge release/1.32.1 into main branch (#668)
* add tabindex to error alerts (#655) * update snapshots (#667)
1 parent 08fbf35 commit a05d024

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@user-interviews/ui-design-system",
3-
"version": "1.32.0",
3+
"version": "1.32.1",
44
"dependencies": {
55
"react-bootstrap": "^2.0.2",
66
"react-router-dom": "^5.2.0",

spec/__snapshots__/Storyshots.test.js.snap

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1286,6 +1286,7 @@ exports[`Storyshots Components/Alert Error 1`] = `
12861286
</div>
12871287
<div
12881288
className="Alert__message"
1289+
tabIndex={-1}
12891290
>
12901291
Error message
12911292
</div>
@@ -1972,6 +1973,7 @@ exports[`Storyshots Components/Alert With Call To Action 1`] = `
19721973
</div>
19731974
<div
19741975
className="Alert__message"
1976+
tabIndex={-1}
19751977
>
19761978
Error message
19771979
</div>

src/Alert/Alert.jsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,13 @@ function Alert(props) {
8585
</div>
8686
)
8787
}
88-
<div className="Alert__message">{props.message}</div>
88+
<div
89+
className="Alert__message"
90+
// eslint-disable-next-line jsx-a11y/no-noninteractive-tabindex
91+
tabIndex={props.type === MessageTypes.ERROR ? -1 : undefined}
92+
>
93+
{props.message}
94+
</div>
8995
</div>
9096
{
9197
props.action && (

0 commit comments

Comments
 (0)