feat(accordion): use appropriate aria tags for accessibility

[jeremytowne]

The id generation in accordion.js, seems a little sketchy - open to ideas about this.

[wesleycho]

Something similar to

bootstrap/src/typeahead/typeahead.js

Line 113 in 3dac5e3

var popupId = 'typeahead-' + scope.$id + '-' + Math.floor(Math.random() * 10000);

would work I think for generating the id attribute - the Math.random usage is not the greatest solution, but is safe IMO.

[jeremytowne]

Ok, I will update it to be similar to that. Should I be adding the aria tags like

bootstrap/src/typeahead/typeahead.js

Line 114 in 3dac5e3

element.attr({

, or is putting them in the template OK?

[wesleycho]

Template is fine

[jeremytowne]

It looks like the travis ci build failed because of a timeout issue, is there anyway to run a new build against this?

[icfantv]

Looks ok now.

[wesleycho]

Can you switch these to - instead of _?

[wesleycho]

Just that switch in those two ids to dashes & this should be good.

[jeremytowne]

Awsome! Thanks for the feedback.