Browse Source

mootools change break context menu

Fx.Tween in mootols 1.4.5  no longer sets visibility to hidden, so when the context menu dissapear, if you put the mouse in the space where the menu were showed is still there but with opacity to 0 and cannot click to nothing below that area

So i emulated the old behaviour with OnComplete event.

Additionally i changed the use of deprecated $empty
adaptive-webui-19844
ccerrillo 12 years ago
parent
commit
c928987313
  1. 14
      src/webui/scripts/contextmenu.js

14
src/webui/scripts/contextmenu.js

@ -27,7 +27,17 @@ var ContextMenu = new Class({ @@ -27,7 +27,17 @@ var ContextMenu = new Class({
this.targets = $$(this.options.targets);
//fx
this.fx = new Fx.Tween(this.menu, { property: 'opacity', duration:this.options.fadeSpeed });
this.fx = new Fx.Tween(this.menu, {
property: 'opacity',
duration:this.options.fadeSpeed,
onComplete: function() {
if(this.getStyle('opacity')){
this.setStyle('visibility','visible');
}else{
this.setStyle('visibility','hidden');
}
}.bind(this.menu)
});
//hide and begin the listener
this.hide().startListener();
@ -158,4 +168,4 @@ var ContextMenu = new Class({ @@ -158,4 +168,4 @@ var ContextMenu = new Class({
return this;
}
});
});

Loading…
Cancel
Save