From 728357707de1e1443b578ca2806c97a09a3caa98 Mon Sep 17 00:00:00 2001
From: Andrew Nicols <andrew@nicols.co.uk>
Date: Wed, 16 Aug 2017 12:32:46 +0800
Subject: [PATCH] MDL-59784 core: Modal factory should listen before fetching
 templates

---
 lib/amd/build/modal_factory.min.js | Bin 1572 -> 1662 bytes
 lib/amd/src/modal_factory.js       |  35 ++++++++++++++++++++---------
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/lib/amd/build/modal_factory.min.js b/lib/amd/build/modal_factory.min.js
index efa71d9cba867366c736d339756638e9dcb50525..02e0604bfb36447227a2224410019bad9340fa81 100644
GIT binary patch
delta 258
zcmZ3&^N(l4b0+1IjMO}hw9>rflFa-(jYQ4rqSTVoqCAB}z2c1gat%$L#M;Skm<owh
zry^~gsE5nQ$?eQ~4nXrXYio7#Y~gk$=_G4bC)*}!Bx_oiB^D{9*e2=a=BJeAr0S&F
z=B1V^q=00g&Ptv9i&=$PC(U-U8jG?;3PcxFE>9y#Cq=VXQ!gzsGe;v+FSQ~$wE*NT
Sxb9qyG@uHdw9UmV8yNvr%vzuT

delta 170
zcmeyzvxH~Eb0*f}jQnzq$@7?1ChIZ#!g=bG?=$PT*4FCe*`}4|C6{F8=V>JAB<rMT
zRwvsgY9wn~mn9Y{q}nFw<>sf9=A`PR+2*B|E2M&Cic(8Ti}Dn5HPV2}b<!q#v#4;u
s1Sc1<C~KucOn|Bb8Jr4KS*xj+mYA8Nk*Swjk(^opvQl&NS(c5A0DogSvH$=8

diff --git a/lib/amd/src/modal_factory.js b/lib/amd/src/modal_factory.js
index 678bad88d33..f9db208d054 100644
--- a/lib/amd/src/modal_factory.js
+++ b/lib/amd/src/modal_factory.js
@@ -55,10 +55,10 @@ define(['jquery', 'core/modal_events', 'core/modal_registry', 'core/modal',
      * is closed.
      *
      * @method setUpTrigger
-     * @param {object} modal The modal instance
+     * @param {Promise} modalPromise The modal instance
      * @param {object} triggerElement The jQuery element to open the modal
      */
-    var setUpTrigger = function(modal, triggerElement) {
+    var setUpTrigger = function(modalPromise, triggerElement) {
         if (typeof triggerElement != 'undefined') {
             if (Array.isArray(triggerElement)) {
                 var selector = triggerElement[1];
@@ -66,19 +66,31 @@ define(['jquery', 'core/modal_events', 'core/modal_registry', 'core/modal',
 
                 CustomEvents.define(triggerElement, [CustomEvents.events.activate]);
                 triggerElement.on(CustomEvents.events.activate, selector, function(e, data) {
-                    modal.show();
+                    modalPromise.then(function(modal) {
+                        modal.show();
+
+                        return modal;
+                    });
                     data.originalEvent.preventDefault();
                 });
             } else {
                 CustomEvents.define(triggerElement, [CustomEvents.events.activate]);
                 triggerElement.on(CustomEvents.events.activate, function(e, data) {
-                    modal.show();
+                    modalPromise.then(function(modal) {
+                        modal.show();
+
+                        return modal;
+                    });
                     data.originalEvent.preventDefault();
                 });
             }
 
-            modal.getRoot().on(ModalEvents.hidden, function() {
-                triggerElement.focus();
+            modalPromise.then(function(modal) {
+                modal.getRoot().on(ModalEvents.hidden, function() {
+                    triggerElement.focus();
+                });
+
+                return modal;
             });
         }
     };
@@ -93,11 +105,10 @@ define(['jquery', 'core/modal_events', 'core/modal_registry', 'core/modal',
      * @param {object} triggerElement The trigger HTML jQuery object
      * @return {object} Modal instance
      */
-    var createFromElement = function(registryConf, modalElement, triggerElement) {
+    var createFromElement = function(registryConf, modalElement) {
         modalElement = $(modalElement);
         var module = registryConf.module;
         var modal = new module(modalElement);
-        setUpTrigger(modal, triggerElement);
 
         return modal;
     };
@@ -115,12 +126,16 @@ define(['jquery', 'core/modal_events', 'core/modal_registry', 'core/modal',
     var createFromType = function(registryConf, templateContext, triggerElement) {
         var templateName = registryConf.template;
 
-        return Templates.render(templateName, templateContext)
+        var modalPromise = Templates.render(templateName, templateContext)
             .then(function(html) {
                 var modalElement = $(html);
-                return createFromElement(registryConf, modalElement, triggerElement);
+                return createFromElement(registryConf, modalElement);
             })
             .fail(Notification.exception);
+
+        setUpTrigger(modalPromise, triggerElement);
+
+        return modalPromise;
     };
 
     /**
-- 
2.43.0