Makes if/return statements uniform to codebase.

This commit is contained in:
zorkow 2015-09-02 14:08:50 +01:00
parent 03c0aac634
commit 03e761dafe

View File

@ -544,16 +544,12 @@
MENU.posted = false; MENU.posted = false;
}, },
MoveHorizontal: function(event, menu, move) { MoveHorizontal: function(event, menu, move) {
if (!event.shiftKey) { return; } if (!event.shiftKey) return;
var jaxs = MENU.AllNodes(); var jaxs = MENU.AllNodes();
var len = jaxs.length; var len = jaxs.length;
if (len === 0) { if (len === 0) return;
return;
}
var next = jaxs[MENU.Mod(move(MENU.IndexOf(jaxs, MENU.CurrentNode())), len)]; var next = jaxs[MENU.Mod(move(MENU.IndexOf(jaxs, MENU.CurrentNode())), len)];
if (next === MENU.CurrentNode()) { if (next === MENU.CurrentNode()) return;
return;
}
MENU.menu.Remove(event, menu); MENU.menu.Remove(event, menu);
MENU.jax = MathJax.Hub.getJaxFor(next); MENU.jax = MathJax.Hub.getJaxFor(next);
MENU.FocusNode(next); MENU.FocusNode(next);
@ -709,9 +705,7 @@
} }
} }
var index = MENU.IndexOf(items, this); var index = MENU.IndexOf(items, this);
if (index === -1) { if (index === -1) return;
return;
}
var len = items.length; var len = items.length;
var children = menuNode.childNodes; var children = menuNode.childNodes;
do { do {
@ -737,7 +731,7 @@
if (menuNode.menuItem === MENU.menu && event.shiftKey) { if (menuNode.menuItem === MENU.menu && event.shiftKey) {
move(event, item); move(event, item);
} }
if (rtl) { return; } if (rtl) return;
if (menuNode.menuItem !== MENU.menu) { if (menuNode.menuItem !== MENU.menu) {
this.Deactivate(item); this.Deactivate(item);
} }
@ -876,9 +870,7 @@
this.SUPER(arguments).MoveHorizontal.apply(this, arguments); this.SUPER(arguments).MoveHorizontal.apply(this, arguments);
return; return;
} }
if (this.disabled) { if (this.disabled) return;
return;
}
if (!this.submenu.posted) { if (!this.submenu.posted) {
this.Activate(event, menu); this.Activate(event, menu);
return; return;