fix #422

Author: basvasilich

blocks/popup/popup.js

@@ -720,7 +720,7 @@ nb.define('popup-toggler', {
         if (evt) {
             evt.preventDefault();
         }
-        if (!this.$node.hasClass('_nb-is-disabled') && this.popup && !this.popup.isOpen()) {
+        if (this.isEnabled() && this.popup && !this.popup.isOpen()) {
             this.popup.open(this.options);
             this.trigger('nb-opened', this);
         }
@@ -733,7 +733,7 @@ nb.define('popup-toggler', {
      * @returns {Object} nb.block
      */
     close: function() {
-        if (!this.$node.hasClass('_nb-is-disabled') && this.popup && this.popup.isOpen()) {
+        if (this.isEnabled() && this.popup && this.popup.isOpen()) {
             this.popup.close();
             this.trigger('nb-closed', this);
         }
@@ -814,6 +814,51 @@ nb.define('popup-toggler', {
         return this;
     },
 
+    /**
+     * Disable the toggler
+     *
+     * ```
+     * popupToggler.disable();
+     * ```
+     *
+     * @fires 'nb-disabled'
+     * @return {Object} blocks for chaining
+     */
+    disable: function() {
+        this._tabindex = this.$node.attr('tabindex');
+        this.$node.attr('tabindex', '-1');
+        this.$node.addClass('_nb-is-disabled');
+        this.trigger('nb-disabled', this);
+        return this;
+    },
+
+    /**
+     * Enables the  toggler
+     *
+     * ```
+     * popupToggler.enable();
+     * ```
+     *
+     * @fires 'nb-enabled'
+     * @return {Object} blocks for chaining
+     */
+    enable: function() {
+        this.$node.attr('tabindex', this._tabindex || '0');
+        this.$node.removeClass('_nb-is-disabled');
+        this.trigger('nb-enabled', this);
+        return this;
+    },
+
+    /**
+     * Return state of the toggler
+     *
+     *
+     * @return {Boolean}
+     */
+    isEnabled: function() {
+        return !this.$node.hasClass('_nb-is-disabled');
+    },
+
     /**
      * Destroy the popup toggler
      * @fires 'nb-destroyed'

blocks/popup/popup.md

@@ -6,7 +6,7 @@ There are 2 parts of popup control:
 #### Options for popup toggler
 
 
-<a `id="popup-toggler" class="nb link link_wrapper link_pseudo" data-nb="popup-toggler" data-nb-popup-toggler="{id: 'popup1'}" href="#default">
+<a id="popup-toggler" class="nb link link_wrapper link_pseudo" data-nb="popup-toggler" data-nb-popup-toggler="{id: 'popup1'}" href="#default">
     <span class="link__inner">Default Toggler</span>
 </a>
 

unittests/spec/popup/popup.js

@@ -140,10 +140,45 @@ describe("Popup Tests", function() {
             expect(this.toggler.getOptions().where).to.equal('.content');
         });
 
-        it("should destroy nb.block", function() {
-            this.toggler.destroy();
-            expect(nb.hasBlock($('#popup-toggler')[0])).to.be.equal(false);
+
+        it("#disable() check state", function() {
+            this.toggler.disable();
+            expect(this.toggler.isEnabled()).to.not.ok();
+        });
+
+        it("#disable() check event", function() {
+            var flag = true;
+
+            this.toggler.on('nb-disabled', function() {
+                flag = false;
+            });
+
+            this.toggler.disable();
+
+            expect(flag).to.not.ok();
+        });
+
+        it("#enable() check state", function() {
+            this.toggler.disable();
+            this.toggler.enable();
+            expect(this.toggler.isEnabled()).to.ok();
         });
+
+        it("#enable() check event", function() {
+            var flag = false;
+            this.toggler.on('nb-enabled', function() {
+                flag = true;
+            });
+
+            this.toggler.disable();
+            this.toggler.enable();
+            expect(flag).to.ok();
+        });
+        it("should destroy nb.block", function() {
+                   this.toggler.destroy();
+                   expect(nb.hasBlock($('#popup-toggler')[0])).to.be.equal(false);
+               });
+
     });
 
     describe("#Popup", function() {