fix(accordion): add CSP compliance for accordion & typeahead

Closes #3909 Fixes #3904
angular-ui · Jul 7, 2015 · fb302c6 · fb302c6
1 parent 158d267
commit fb302c6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/template/accordion/accordion-group.html b/template/accordion/accordion-group.html
@@ -1,7 +1,7 @@
 <div class="panel panel-default">
   <div class="panel-heading">
     <h4 class="panel-title">
-      <a href="javascript:void(0)" tabindex="0" class="accordion-toggle" ng-click="toggleOpen()" accordion-transclude="heading"><span ng-class="{'text-muted': isDisabled}">{{heading}}</span></a>
+      <a href="#" tabindex="0" class="accordion-toggle" ng-click="$event.preventDefault(); toggleOpen()" accordion-transclude="heading"><span ng-class="{'text-muted': isDisabled}">{{heading}}</span></a>
     </h4>
   </div>
   <div class="panel-collapse collapse" collapse="!isOpen">

diff --git a/template/typeahead/typeahead-match.html b/template/typeahead/typeahead-match.html
@@ -1 +1 @@
-<a href="javascript:void(0)" tabindex="-1" bind-html-unsafe="match.label | typeaheadHighlight:query"></a>
+<a href="#" ng-click="$event.preventDefault()" tabindex="-1" bind-html-unsafe="match.label | typeaheadHighlight:query"></a>
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		<a href="javascript:void(0)" tabindex="-1" bind-html-unsafe="match.label \| typeaheadHighlight:query"></a>
		<a href="#" ng-click="$event.preventDefault()" tabindex="-1" bind-html-unsafe="match.label \| typeaheadHighlight:query"></a>