Skip to content

E
edx-platform
Commit 71590419 by Prem Sichanugrist Committed by Matthew Mongeau
Options

Be specific on video progress slider handle 

Using `.ui-slider-handle` was too broad.
parent 2666f9bf
Showing with 2 additions and 2 deletions
lms/static/coffee/spec/modules/video/video_progress_slider_spec.coffee
...@@ -18,7 +18,7 @@ describe 'VideoProgressSlider', -> ...@@ -18,7 +18,7 @@ describe 'VideoProgressSlider', ->
stop: @slider.onStop stop: @slider.onStop
it 'build the seek handle', -> it 'build the seek handle', ->
expect(@slider.handle).toBe '.ui-slider-handle' expect(@slider.handle).toBe '.slider .ui-slider-handle'
expect($.fn.qtip).toHaveBeenCalledWith expect($.fn.qtip).toHaveBeenCalledWith
content: "0:00" content: "0:00"
position: position:
......
lms/static/coffee/src/modules/video/video_progress_slider.coffee
...@@ -17,7 +17,7 @@ class @VideoProgressSlider ...@@ -17,7 +17,7 @@ class @VideoProgressSlider
@buildHandle() @buildHandle()
buildHandle: -> buildHandle: ->
@handle = @$('.ui-slider-handle') @handle = @$('.slider .ui-slider-handle')
@handle.qtip @handle.qtip
content: "#{Time.format(@slider.slider('value'))}" content: "#{Time.format(@slider.slider('value'))}"
position: position:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment