Skip to content

Commit 8c10c1e

Browse files
committed
Revert "Selectmenu: added a temp fix for Safari to make unit test work, see jquery#492 (comment)"
This reverts commit 72c17f0.
1 parent 72c17f0 commit 8c10c1e

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

tests/unit/selectmenu/selectmenu_core.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ $.each([
6060
equals( element.find("option:selected").val(), selected.next("option").val() , "after keydown original select state" );
6161
equals( button.text(), selected.next("option").text(), "after keydown button text" );
6262

63-
button.simulate( "click", { clientX: 1, clientY: 1 } );
64-
menu.find("a").last().simulate( "mouseover", { clientX: 1, clientY: 1 } ).trigger( "click" );
63+
button.simulate( "click" );
64+
menu.find("a").last().simulate( "mouseover" ).trigger( "click" );
6565
equals( menu.attr("aria-activedescendant"), links.eq(element[0].selectedIndex).attr("id"), "after click menu aria-activedescendant" );
6666
equals( links.eq(element[0].selectedIndex).attr("aria-selected"), "true", "after click selected menu link aria-selected" );
6767
equals( element.find("option:selected").val(), element.find("option").last().val(), "after click original select state" );

tests/unit/selectmenu/selectmenu_events.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ test("change", function () {
2323
menu = this.element.selectmenu("menuWidget").parent(),
2424
value = this.element.find("option").first().text();
2525

26-
button.find("a").simulate( "focus" ).simulate( "click", { clientX: 1, clientY: 1 } );
27-
menu.find("a").first().simulate( "mouseover", { clientX: 1, clientY: 1 } ).trigger( "click" );
26+
button.find("a").simulate( "focus" ).simulate( "click" );
27+
menu.find("a").first().simulate( "mouseover" ).trigger( "click" );
2828
});
2929

3030

@@ -63,8 +63,8 @@ test("focus", function () {
6363
var button = this.element.selectmenu("widget").parent(),
6464
menu = this.element.selectmenu("menuWidget").parent();
6565

66-
button.find("a").simulate( "focus" ).simulate( "click", { clientX: 1, clientY: 1 } );
67-
menu.find(".ui-menu-item").simulate("mouseover", { clientX: 1, clientY: 1 });
66+
button.find("a").simulate( "focus" ).simulate( "click" );
67+
menu.find(".ui-menu-item").simulate("mouseover");
6868
});
6969

7070

@@ -98,8 +98,8 @@ test("select", function () {
9898
var button = this.element.selectmenu("widget").parent(),
9999
menu = this.element.selectmenu("menuWidget").parent();
100100

101-
button.find("a").simulate( "focus" ).simulate( "click", { clientX: 1, clientY: 1 } );
102-
menu.find("a").first().simulate( "mouseover", { clientX: 1, clientY: 1 } ).trigger("click");
101+
button.find("a").simulate( "focus" ).simulate( "click" );
102+
menu.find("a").first().simulate( "mouseover" ).trigger("click");
103103
});
104104

105105
})(jQuery);

0 commit comments

Comments
 (0)