Merge pull request #66 from FinalAngel/bugfix/mobile-menu-overlay

move overlay initialization

index.html

@@ -18,7 +18,7 @@
   - name: lightbox
     version: 1.2.2
   - name: mobilemenu
-    version: 1.1.6
+    version: 1.1.7
   - name: uniform
     version: 1.2.3
 

src/cl.mobilemenu/CHANGELOG.rst

@@ -2,6 +2,10 @@
 Cl.MobileMenu
 =============
 
+1.1.7
+-----
+- moved overlay initialization from ``toggle`` to ``show``
+
 1.1.6
 -----
 - added option ``heightCalc``
@@ -56,4 +60,4 @@ Cl.MobileMenu
 
 1.0.0
 -----
-- initial release
+- initial release

src/cl.mobilemenu/cl.mobilemenu.js

@@ -1,7 +1,7 @@
 /*!
  * @author      Angelo Dini - github.com/finalangel/classjs-plugins
  * @copyright   Distributed under the BSD License.
- * @version     1.1.6
+ * @version     1.1.7
  */
 
 // ensure namespace is defined
@@ -81,15 +81,15 @@ var Cl = window.Cl || {};
         toggle: function () {
             if(this.visible) { this.hide(); } else { this.show(); }
 
-            // if not initialized, inject overlay
-            if(!this.initialized) $(this.options.overlayContainer).append(this.overlay);
-            this.initialized = true;
-
             // trigger callback
             this._fire('toggle');
         },
 
         show: function (speed) {
+            // if not initialized, inject overlay
+            if(!this.initialized) $(this.options.overlayContainer).append(this.overlay);
+            this.initialized = true;
+
             // validate if toolbar should be shown
             if(!this._validate()) return false;
 
@@ -183,4 +183,4 @@ var Cl = window.Cl || {};
 
     });
 
-})(jQuery);
+})(jQuery);