Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed target used for span event.

Browse Source
This commit is contained in:
Arthur Barrett
2013-02-04 18:53:37 -05:00
parent 435fd05ef4
commit 3136d2b4c1
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
common/lib/xmodule/xmodule/js/src/annotatable/display.coffee
View File

@@ -32,15 +32,17 @@ class @Annotatable
onSpanEvent: (fn) ->
(e) =>
span_id = e.target.getAttribute('data-span-id')
span_el = e.currentTarget
span_id = span_el.getAttribute('data-span-id')
discussion_id = @getDiscussionId(span_id)
discussion_el = @getDiscussionEl(discussion_id)
span = {
id: span_id,
el: e.target
id: span_id
el: span_el
}
discussion = {
id: discussion_id,
el: @getDiscussionEl(discussion_id)
id: discussion_id
el: discussion_el
}
fn.call this, span, discussion
@@ -48,10 +50,13 @@ class @Annotatable
@scrollToDiscussion(discussion.el)
onEnterSpan: (span, discussion) ->
$(@discussionSelector, @el).not(discussion.el).toggleClass('opaque', true)
@focusDiscussion(discussion.el, true)
onLeaveSpan: (span, discussion) ->
$(@discussionSelector, @el).not(discussion.el).toggleClass('opaque', false)
@focusDiscussion(discussion.el, false)
focusDiscussion: (el, state) ->
$(@discussionSelector, @el).not(el).toggleClass('opaque', state)
scrollToDiscussion: (el) ->
complete = @makeHighlighter(el)