Shouldn't this have role="button"? Also, shouldn't we be auto-generating some of this versus hard-coding it in the markup?