refactor(modal): set alert dialog attributes in markup (#1333)

This commit is contained in:
metonym 2022-06-02 18:41:18 -07:00 committed by GitHub
commit e62bea38aa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -139,16 +139,6 @@
$: ariaLabel =
modalLabel || $$props["aria-label"] || modalAriaLabel || modalHeading;
let alertDialogProps = {};
$: if (alert) {
if (passiveModal) {
alertDialogProps.role = "alert";
}
if (!passiveModal) {
alertDialogProps.role = "alertdialog";
alertDialogProps["aria-describedby"] = modalBodyId;
}
}
$: if (open) {
addModalId(id);
} else {
@ -214,9 +204,9 @@
>
<div
bind:this="{innerModal}"
role="dialog"
tabindex="-1"
{...alertDialogProps}
role="{alert ? (passiveModal ? 'alert' : 'alertdialog') : 'dialog'}"
aria-describedby="{alert && !passiveModal ? modalBodyId : undefined}"
aria-modal="true"
aria-label="{ariaLabel}"
class:bx--modal-container="{true}"