Skip to content

Commit dd28e40

Browse files
authored
Merge pull request desktop#17519 from desktop/alertdialog-for-local-changes-error
Use `role=alertdialog` for errors when trying to do operations with local changes
2 parents 57180aa + f769165 commit dd28e40

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

app/src/ui/local-changes-overwritten/local-changes-overwritten-dialog.tsx

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,14 +62,18 @@ export class LocalChangesOverwrittenDialog extends React.Component<
6262
onDismissed={this.props.onDismissed}
6363
onSubmit={this.onSubmit}
6464
type="error"
65+
role="alertdialog"
66+
ariaDescribedBy="local-changes-error-description"
6567
>
6668
<DialogContent>
67-
<p>
68-
Unable to {this.getRetryActionName()} when changes are present on
69-
your branch.{overwrittenText}
70-
</p>
71-
{this.renderFiles()}
72-
{this.renderStashText()}
69+
<div id="local-changes-error-description">
70+
<p>
71+
Unable to {this.getRetryActionName()} when changes are present on
72+
your branch.{overwrittenText}
73+
</p>
74+
{this.renderFiles()}
75+
{this.renderStashText()}
76+
</div>
7377
</DialogContent>
7478
{this.renderFooter()}
7579
</Dialog>

0 commit comments

Comments
 (0)