enable scroll when isOpen is set to false

material-components · May 15, 2018 · 7150bd3 · 7150bd3
1 parent edce36d
commit 7150bd3
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/packages/mdc-dialog/foundation.js b/packages/mdc-dialog/foundation.js
@@ -76,8 +76,8 @@ export default class MDCDialogFoundation extends MDCFoundation {
       this.adapter_.deregisterTransitionEndHandler(this.transitionEndHandler_);
       this.adapter_.removeClass(MDCDialogFoundation.cssClasses.ANIMATING);
       this.adapter_.removeClass(MDCDialogFoundation.cssClasses.OPEN);
+      this.enableScroll_();
     }
-    this.enableScroll_();
   }
 
   open() {
@@ -93,6 +93,7 @@ export default class MDCDialogFoundation extends MDCFoundation {
 
   close() {
     this.isOpen_ = false;
+    this.enableScroll_();
     this.adapter_.deregisterSurfaceInteractionHandler('click', this.dialogClickHandler_);
     this.adapter_.deregisterDocumentKeydownHandler(this.documentKeydownHandler_);
     this.adapter_.deregisterInteractionHandler('click', this.componentClickHandler_);
@@ -137,8 +138,6 @@ export default class MDCDialogFoundation extends MDCFoundation {
       this.adapter_.removeClass(MDCDialogFoundation.cssClasses.ANIMATING);
       if (this.isOpen_) {
         this.adapter_.trapFocusOnSurface();
-      } else {
-        this.enableScroll_();
       };
     };
   };