Merge pull request #52 from DanielApt/patch-2

Fix for incorrect limit with remote

Author: jcrben

src/typeahead/dataset.js

@@ -269,10 +269,10 @@ var Dataset = (function() {
         // do not render the suggestions as they've become outdated
         if (!canceled && rendered < that.limit) {
           that.cancel = $.noop;
-          that._append(query, suggestions.slice(0, that.limit - rendered));
-          rendered += suggestions.length;
-
-          that.async && that.trigger('asyncReceived', query);
+          var idx = Math.abs(rendered - that.limit);
+          rendered += idx;
+          that._append(query, suggestions.slice(0, idx));
+          that.async && that.trigger("asyncReceived", query);
         }
       }
     },

test/typeahead/dataset_spec.js

@@ -225,6 +225,7 @@ describe('Dataset', function() {
     });
 
     it('should respect limit option in regard to async', function() {
+      this.dataset.limit = 6;
       this.dataset.async = true;
       this.source.andCallFake(fakeGetWithAsyncSuggestions);
 
@@ -233,7 +234,7 @@ describe('Dataset', function() {
       waits(100);
 
       runs(function() {
-        expect(this.dataset.$el.find('.tt-suggestion')).toHaveLength(5);
+        expect(this.dataset.$el.find('.tt-suggestion')).toHaveLength(6);
       });
     });