link_pseudo: Normalize keyboard interaction #932

bem · Oct 30, 2014 · 45265ea · 45265ea
1 parent 7a9136e
commit 45265ea
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 3 deletions.
diff --git a/common.blocks/link/_pseudo/link_pseudo.js b/common.blocks/link/_pseudo/link_pseudo.js
@@ -2,17 +2,37 @@
  * @module link
  */
 
-modules.define('link', function(provide, Link) {
+modules.define('link', ['keyboard__codes'], function(provide, keyCodes, Link) {
 
 /**
  * @exports
  * @class link
  * @bem
  */
 provide(Link.decl({ modName : 'pseudo', modVal : true }, /** @lends link.prototype */{
+    onSetMod : {
+        'focused' : {
+            'true' : function() {
+                this.__base.apply(this, arguments);
+
+                this.bindTo('control', 'keydown', this._onKeyDown);
+            },
+            '' : function() {
+                this.__base.apply(this, arguments);
+
+                this.unbindFrom('control', 'keydown', this._onKeyDown);
+            }
+        }
+    },
+
     _onPointerClick : function(e) {
         e.preventDefault();
+
         this.__base.apply(this, arguments);
+    },
+
+    _onKeyDown : function(e) {
+        e.keyCode === keyCodes.ENTER && this._onPointerClick(e);
     }
 }));
 

diff --git a/common.blocks/link/_pseudo/link_pseudo.spec.js b/common.blocks/link/_pseudo/link_pseudo.spec.js
@@ -1,7 +1,7 @@
 modules.define(
     'spec',
-    ['link', 'i-bem__dom', 'jquery', 'BEMHTML'],
-    function(provide, Link, BEMDOM, $, BEMHTML) {
+    ['link', 'i-bem__dom', 'jquery', 'BEMHTML', 'sinon', 'keyboard__codes'],
+    function(provide, Link, BEMDOM, $, BEMHTML, sinon, keyCodes) {
 
 describe('link_pseudo', function() {
     var link;
@@ -25,6 +25,16 @@ describe('link_pseudo', function() {
         link.domElem.trigger(e);
         e.isDefaultPrevented().should.be.true;
     });
+
+    it('should emit click on "enter" key when focused', function() {
+        var spy = sinon.spy();
+
+        link.setMod('focused');
+        link.on('click', spy);
+        link.domElem.trigger($.Event('keydown', { keyCode : keyCodes.ENTER }));
+
+        spy.should.have.been.calledOnce;
+    });
 });
 
 provide();

diff --git a/common.blocks/link/link.deps.js b/common.blocks/link/link.deps.js
@@ -5,6 +5,7 @@
     shouldDeps : [
         { mod : 'disabled' },
         { block : 'jquery', elem : 'event', mods : { type : 'pointer' } },
+        { block : 'keyboard', elem : 'codes' },
         'control'
     ]
 },