Merge pull request #282 from canjs/273-dont-autofocus-search

Don't auto focus input and fix tab indices. fixes #273

Author: imaustink

static/search.js

@@ -88,12 +88,6 @@ var Search = Control.extend({
 					self.$inputWrap.show();
 				}
 
-				//focus the search on init
-				//only do stuff if we have an input to work with
-				if(self.$input && self.$input.length){
-					self.$input.trigger("focus");
-				}
-
 				self.bindResultsEvents();
 			}, function(error){
 				console.error("getSearchMap error", error);
@@ -483,26 +477,6 @@ var Search = Control.extend({
 	},
 	// ---- END RESULTS EVENTS ---- //
 
-
-	// ---- WINDOW EVENTS ---- //
-
-	// [ctrl + k] focuses the search input
-	"{window} keyup": function(el, ev){
-		if(!ev.ctrlKey){
-			return true;
-		}
-
-		switch(ev.keyCode){
-			case 75: // 'k'
-				this.$input.trigger("focus");
-				break;
-		}
-	},
-
-	// ---- END WINDOW EVENTS ---- //
-
-
-
 	// ---- SEARCH VIEW ---- //
 
 	// function search

templates/search-bar.mustache

@@ -2,6 +2,13 @@
 	<div class="search-wrap" style="display:none;">
 		<span class="search-icon"></span>
 		<span class="search-icon-cancel"></span>
-		<input type="text" class="search" placeholder="Search - [ctrl + k]" autocomplete="off" autocorrect="off" autocapitalize="none" spellcheck="false" />
+		<input
+			type="text"
+			class="search"
+			placeholder="Search"
+			autocomplete="off"
+			autocorrect="off"
+			autocapitalize="none"
+			spellcheck="false"/>
 	</div>
 </div>