Remove smartSpaceBar option from typeahead and search.

The option caused some race conditions on Firefox, and it is
really made moot by the naturalSearch option anyway.

(imported from commit dc7080c905ced9b2f4ad4275d82549acf09a59f7)
This commit is contained in:
Steve Howell 2013-07-29 13:41:46 -04:00
parent 4b94b99c17
commit 546ae1023c
2 changed files with 1 additions and 23 deletions

View File

@ -485,8 +485,7 @@ exports.initialize = function () {
updater: narrow_or_search_for_term, updater: narrow_or_search_for_term,
sorter: function (items) { sorter: function (items) {
return items; return items;
}, }
smartSpaceBar: true
}); });
$("#searchbox_form").keydown(function (e) { $("#searchbox_form").keydown(function (e) {

View File

@ -1849,19 +1849,6 @@
this.$element.val(val) this.$element.val(val)
} }
, select_and_continue: function () {
// We don't call updater() because it potentially has side-effects.
// This does mean that smartSpaceBar can't be used in cases where the
// item values are transformed by updater(). It's generally imprudent
// to do transformations in updater(), so this should be a
// non-issue for most users.
var val = this.$menu.find('.active').data('typeahead-value')
this.$element
.val(val + ' ')
.change()
return this.hide()
}
, updater: function (item) { , updater: function (item) {
return item return item
} }
@ -2058,14 +2045,6 @@
this.select() this.select()
break break
case 32: // space
if (this.options.smartSpaceBar) {
if (!this.shown) return
this.select_and_continue()
}
this.lookup()
break
case 27: // escape case 27: // escape
if (!this.shown) return if (!this.shown) return
this.hide() this.hide()