Fixing #9: hitting tab at the end of the search box's last input with…

… no text just sends the cursor on its way and no longer loops around to the first input.

lib/js/views/search_box.js

@@ -325,8 +325,7 @@ VS.ui.SearchBox = Backbone.View.extend({
     } else if (options.skipToFacet && currentView.type == 'text' &&
                viewCount == viewPosition && direction >= 0) {
       // Special case of looping around to a facet from the last search input box.
-      viewPosition = 0;
-      direction    = 0;
+      return false;
     }
     var view, next = Math.min(viewCount, viewPosition + direction);
 
@@ -359,6 +358,8 @@ VS.ui.SearchBox = Backbone.View.extend({
     }
     if (options.selectText) view.selectText();
     this.resizeFacets();
+
+    return true;
   },
 
   maybeFocusSearch : function(e) {

lib/js/views/search_input.js

@@ -363,19 +363,22 @@ VS.ui.SearchInput = Backbone.View.extend({
       e.preventDefault();
       this.app.searchBox.focusNextFacet(this, -1, {selectText: true});
     } else if (key == 'tab') {
-      e.preventDefault();
       var value = this.box.val();
       if (value.length) {
+        e.preventDefault();
         var remainder = this.addTextFacetRemainder(value);
         var position  = this.options.position + (remainder?1:0);
         if (value != remainder) {
             this.app.searchBox.addFacet(value, '', position);
         }
       } else {
-        this.app.searchBox.focusNextFacet(this, 0, {
+        var foundFacet = this.app.searchBox.focusNextFacet(this, 0, {
           skipToFacet: true,
           selectText: true
         });
+        if (foundFacet) {
+          e.preventDefault();
+        }
       }
     } else if (VS.app.hotkeys.command &&
                String.fromCharCode(e.which).toLowerCase() == 'a') {