Merge pull request #185 from dfrencham/accessibility-template

Apply accessibility attributes to suggestion template

Author: jlbooker

src/typeahead/dataset.js

@@ -319,9 +319,14 @@ var Dataset = (function() {
       pending: templates.pending && _.templatify(templates.pending),
       header: templates.header && _.templatify(templates.header),
       footer: templates.footer && _.templatify(templates.footer),
-      suggestion: templates.suggestion || suggestionTemplate
+      suggestion: templates.suggestion ? userSuggestionTemplate : suggestionTemplate
     };
 
+    function userSuggestionTemplate(context) {
+      var template = templates.suggestion;
+      return $(template(context)).attr("id", _.guid());
+    }
+
     function suggestionTemplate(context) {
       return $('<div role="option">').attr('id', _.guid()).text(displayFn(context));
     }

test/typeahead/dataset_spec.js

@@ -421,6 +421,33 @@ describe('Dataset', function() {
     });
   });
 
+  it('should apply id attribute when using default suggestion template', function() {
+    this.dataset = new Dataset({
+      source: this.source,
+      node: $('<div>'),
+    }, www);
+
+    this.source.andCallFake(syncMockSuggestions);
+    this.dataset.update('woah');
+
+    expect(this.dataset.$el.find('div[role="option"]')).toHaveAttr('id');
+  });
+
+  it('should apply id attribute when using custom suggestion template', function() {
+    this.dataset = new Dataset({
+      source: this.source,
+      node: $('<div>'),
+      templates: {
+        suggestion: function(c) { return '<p class="search-result"> result' + c.query + '</p>'; }
+      }
+    }, www);
+
+    this.source.andCallFake(syncMockSuggestions);
+    this.dataset.update('woah');
+
+    expect(this.dataset.$el.find('p.search-result')).toHaveAttr('id');
+  });
+
   describe('#clear', function() {
     it('should clear suggestions', function() {
       this.source.andCallFake(syncMockSuggestions);