Skip to content

Commit ee34b0d

Browse files
committed
Menu: Get rid of .css(top:0,left:0) workaround, not necessary anymore
1 parent d74fb20 commit ee34b0d

File tree

4 files changed

+2
-15
lines changed

4 files changed

+2
-15
lines changed

tests/visual/menu/drilldown.html

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,6 @@
5959

6060
_open: function(submenu) {
6161
this.active = submenu.show().css({
62-
top: 0,
63-
left: 0,
6462
opacity: 0
6563
}).position({
6664
my: "left top",

tests/visual/menu/flyoutmenu.html

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@
2929
event.stopPropagation();
3030
menu.flyoutmenu("show")
3131
.focus()
32-
.css({
33-
top: 0,
34-
left: 0
35-
})
3632
.position({
3733
my: "left top",
3834
at: "right top",

tests/visual/menu/flyoutmenu.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,10 +106,7 @@ $.widget("ui.flyoutmenu", {
106106
: this.options.position
107107
);
108108

109-
submenu.show().css({
110-
top: 0,
111-
left: 0
112-
}).position(position);
109+
submenu.show().position(position);
113110
},
114111
_select: function(event) {
115112
event.stopPropagation();

tests/visual/menu/menubar.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,7 @@ $.widget("ui.menubar", {
5555

5656
_open: function(event, menu) {
5757
this.active && this.active.flyoutmenu("hide");
58-
this.active = menu.flyoutmenu("show").css({
59-
position: "absolute",
60-
top: 0,
61-
left: 0
62-
}).position({
58+
this.active = menu.flyoutmenu("show").position({
6359
my: "left top",
6460
at: "left bottom",
6561
offset: "0 -1",

0 commit comments

Comments
 (0)