From a68d5ca31d764a737653461cfba49debdc8ad0ba Mon Sep 17 00:00:00 2001 From: Kris Borchers Date: Sat, 24 Nov 2012 23:18:51 -0600 Subject: [PATCH] Dialog: Add unit test to cover #8789 and #8838. --- tests/unit/dialog/dialog_events.js | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/tests/unit/dialog/dialog_events.js b/tests/unit/dialog/dialog_events.js index eeb17eb93..cbeced0f8 100644 --- a/tests/unit/dialog/dialog_events.js +++ b/tests/unit/dialog/dialog_events.js @@ -325,4 +325,20 @@ test("beforeClose", function() { el.remove(); }); +// #8789 and #8838 +asyncTest("ensure dialog's container doesn't scroll on resize and focus", function() { + expect(2); + + var el = $('#dialog1').dialog(), + initialScroll = $(window).scrollTop(); + el.dialog('option', 'height', 600); + equal($(window).scrollTop(), initialScroll, "scroll hasn't moved after height change"); + setTimeout( function(){ + $(".ui-dialog-titlebar-close").simulate('mousedown'); + equal($(window).scrollTop(), initialScroll, "scroll hasn't moved after focus moved to dialog"); + el.dialog('destroy'); + start(); + }, 500); +}); + })(jQuery);