Build only first time lightbox is opened (ampproject#5941)

Build only first time lightbox is opened.
Lith · Dec 22, 2016 · a85e662 · a85e662
1 parent ac1dfa6
commit a85e662
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/extensions/amp-lightbox/0.1/amp-lightbox.js b/extensions/amp-lightbox/0.1/amp-lightbox.js
@@ -47,8 +47,15 @@ class AmpLightbox extends AMP.BaseElement {
     return layout == Layout.NODISPLAY;
   }
 
-  /** @override */
-  buildCallback() {
+  /**
+   * Lazily builds the lightbox DOM on the first open.
+   * @private
+   */
+  initialize_() {
+    if (this.container_) {
+      return;
+    }
+
     st.setStyles(this.element, {
       position: 'fixed',
       zIndex: 1000,
@@ -86,6 +93,7 @@ class AmpLightbox extends AMP.BaseElement {
     if (this.active_) {
       return;
     }
+    this.initialize_();
     this.boundCloseOnEscape_ = this.closeOnEscape_.bind(this);
     this.win.document.documentElement.addEventListener(
         'keydown', this.boundCloseOnEscape_);