From 411e5f4e33fddb88d70299a7b4d2d8b66f2593a0 Mon Sep 17 00:00:00 2001 From: Ari Entlich Date: Wed, 30 Apr 2014 15:00:59 -0400 Subject: [PATCH] Fix clicking on parts of annotations that are in a different element than where the annotation starts. Fixes #20. --- app/assets/javascripts/sidebar.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/sidebar.js b/app/assets/javascripts/sidebar.js index f805f85d0..759a944e3 100644 --- a/app/assets/javascripts/sidebar.js +++ b/app/assets/javascripts/sidebar.js @@ -130,9 +130,8 @@ Sidebar.AnnotationListView = Backbone.View.extend({ $("span.annotator-hl").click(function(event) { $("ul#annotation-list li").removeClass('hover'); $("span.highlightlink").tooltip('hide'); - var str = this.id.toString(); - var parts = str.match(/(hl)(.+)/).slice(1); - var targetid = "#sb" + parts[1]; + var uuid = $(this).data('annotation').uuid; + var targetid = "#sb" + uuid; // TODO: deal with the events in a more organized way (recompose them in functions) $('div#annotation-well').animate({scrollTop:$(targetid).offset().top}, 100, function (){