Commit 01605bc7 authored by Mike Greiling's avatar Mike Greiling

fix body scrollTop references

parent 80c25e8b
...@@ -192,7 +192,7 @@ feature 'Diff notes resolve', :js do ...@@ -192,7 +192,7 @@ feature 'Diff notes resolve', :js do
page.find('.discussion-next-btn').click page.find('.discussion-next-btn').click
end end
expect(page.evaluate_script("$('body').scrollTop()")).to be > 0 expect(page.evaluate_script("window.pageYOffset")).to be > 0
end end
it 'hides jump to next button when all resolved' do it 'hides jump to next button when all resolved' do
...@@ -308,7 +308,7 @@ feature 'Diff notes resolve', :js do ...@@ -308,7 +308,7 @@ feature 'Diff notes resolve', :js do
page.find('.discussion-next-btn').click page.find('.discussion-next-btn').click
end end
expect(page.evaluate_script("$('body').scrollTop()")).to be > 0 expect(page.evaluate_script("window.pageYOffset")).to be > 0
end end
it 'updates updated text after resolving note' do it 'updates updated text after resolving note' do
......
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