fixup! feat(material-experimental): MDC-based version of dialog

Address feedback

Author: devversion

src/material-experimental/mdc-dialog/_dialog-legacy-padding.scss

@@ -26,8 +26,8 @@ $mat-mdc-dialog-legacy-padding: 24px !default;
     }
 
     .mat-mdc-dialog-title, .mat-mdc-dialog-content, .mat-mdc-dialog-actions {
-      margin-left: $mat-mdc-dialog-legacy-padding * -1;
-      margin-right: $mat-mdc-dialog-legacy-padding * -1;
+      margin-left: -$mat-mdc-dialog-legacy-padding;
+      margin-right: -$mat-mdc-dialog-legacy-padding;
     }
   }
 }

src/material-experimental/mdc-dialog/dialog-container.ts

@@ -38,7 +38,7 @@ import {cssClasses, numbers} from '@material/dialog';
     'class': 'mat-mdc-dialog-container mdc-dialog',
     'tabindex': '-1',
     'aria-modal': 'true',
-    '[attr.id]': '_id',
+    '[id]': '_id',
     '[attr.role]': '_config.role',
     '[attr.aria-labelledby]': '_config.ariaLabel ? null : _ariaLabelledBy',
     '[attr.aria-label]': '_config.ariaLabel',
@@ -85,8 +85,9 @@ export class MatDialogContainer extends _MatDialogContainerBase implements
     //   1. Foundation does not allow us to disable animations.
     //   2. Foundation contains unnecessary features we don't need and aren't
     //      tree-shakeable. e.g. background scrim, keyboard event handlers for ESC button.
-    //   3. Foundation uses unnecessary timers for animations in favor of React `setState`
-    //      limitations: https://github.com/material-components/material-components-web/pull/3682.
+    //   3. Foundation uses unnecessary timers for animations to work around limitations
+    //      in React's `setState` mechanism.
+    //      https://github.com/material-components/material-components-web/pull/3682.
     this._startOpenAnimation();
   }
 

src/material/dialog/dialog-container.ts

@@ -238,7 +238,7 @@ export abstract class _MatDialogContainerBase extends BasePortalOutlet implement
     'class': 'mat-dialog-container',
     'tabindex': '-1',
     'aria-modal': 'true',
-    '[attr.id]': '_id',
+    '[id]': '_id',
     '[attr.role]': '_config.role',
     '[attr.aria-labelledby]': '_config.ariaLabel ? null : _ariaLabelledBy',
     '[attr.aria-label]': '_config.ariaLabel',