Fixes

* Also update the ng-model value on **de**select.
* kbItem disables tabbing to :tabbable elements.
  (Ensures using tab between lists and arrow keys between items)

Author: bfanger

src/directives/KbContainerController.js

@@ -77,10 +77,19 @@ angular.module('keyboard').factory('KbContainerController', function (undefined,
          * @param {*} model
          */
         deselect: function (model) {
+            if (!this.ngModel) {
+                return;
+            }
             var index = this.selected.indexOf(model);
             if (index !== -1) {
                 this.selected.splice(index, 1);
+                if (this.multiple) {
+                    this.ngModel.$setViewValue(this.selected);
+                } else {
+                    this.ngModel.$setViewValue(undefined);
+                }
             }
+
         },
         /**
          * Select or deselect the given model.

src/directives/kbItem.js

@@ -9,7 +9,7 @@ angular.module('keyboard').directive('kbItem', function (KbItemController, $anim
         link: function ($scope, el, attrs, controllers) {
             var kbItem = controllers[0];
             var kbContainer = controllers[1];
-            var tagName = el[0].tagName;
+            var isTabbable = (el[0].tagName === 'A' || el[0].tagName === 'BUTTON' || el[0].tagName === 'AREA');
 
             for (var i = 1; i < controllers.length; i++) {
                 if (controllers[i]) {
@@ -52,8 +52,8 @@ angular.module('keyboard').directive('kbItem', function (KbItemController, $anim
                     $animate.addClass(el, activeClass);
                 } else {
                     $animate.removeClass(el, activeClass);
-                    if (tagName === 'A' || tagName === 'BUTTON') {
-                        el.attr('tabindex', 0);
+                    if (isTabbable) {
+                        el.attr('tabindex', -1);
                     } else {
                         el.removeAttr('tabindex');
                     }