Avoid scrolling the page when expanding/collapsing excerpts

Stop propagation of the click event in the 'More'/'Less'
links which would otherwise bubble up to the
annotation card and scroll the page to that annotation.

h/static/scripts/directive/annotation.js

@@ -587,6 +587,11 @@ function AnnotationController(
     }
   };
 
+  vm.toggleCollapseBody = function(event) {
+    event.stopPropagation();
+    vm.collapseBody = !vm.collapseBody;
+  };
+
   /**
     * @ngdoc method
     * @name annotation.AnnotationController#reply

h/static/scripts/directive/excerpt.js

@@ -21,7 +21,8 @@ function ExcerptController() {
     return this.overflowing() && !this.collapse;
   };
 
-  this.toggle = function () {
+  this.toggle = function (event) {
+    event.stopPropagation();
     this.collapse = !this.collapse;
   };
 

h/templates/client/annotation.html

@@ -109,7 +109,7 @@
     </ul>
     <div class="u-stretch"></div>
     <a class="annotation-link u-strong" ng-show="vm.canCollapseBody"
-       ng-click="vm.collapseBody = !vm.collapseBody"
+       ng-click="vm.toggleCollapseBody($event)"
        ng-title="vm.collapseBody ? 'Show the full annotation text' : 'Show the first few lines only'"
        ng-bind="vm.collapseBody ? 'More' : 'Less'"></a>
   </div>

h/templates/client/excerpt.html

@@ -9,12 +9,12 @@
             title="Show the full excerpt">More</a>
       </span>
       <span class="excerpt__toggle-link" ng-show="vm.isCollapsible()">
-        <a ng-click="vm.toggle()"
+        <a ng-click="vm.toggle($event)"
             title="Show the first few lines only">Less</a>
       </span>
     </div>
   </div>
-  <div ng-click="vm.toggle()"
+  <div ng-click="vm.toggle($event)"
        ng-class="vm.bottomShadowStyles()"
        title="Show the full excerpt"></div>
 </div>