[refactor] improve modal focus

Author: zombieFox

src/js/menu.js

@@ -1,31 +1,5 @@
 varmenu=(function(){
 
-varbind={};
-
-bind.focus={
-loop: function(event){
-varfirstElement=helper.e(".control-menu-layout");
-varlastElement=helper.e(".menu-close-tab");
-if(event.keyCode==9&&event.shiftKey){
-if(document.activeElement===firstElement){
-lastElement.focus();
-event.preventDefault();
-}
-}elseif(event.keyCode==9){
-if(document.activeElement===lastElement){
-firstElement.focus();
-event.preventDefault();
-}
-};
-},
-add: function(){
-window.addEventListener("keydown",bind.focus.loop,false);
-},
-remove: function(){
-window.removeEventListener("keydown",bind.focus.loop,false);
-}
-};
-
 varmod={};
 
 mod.open=function(){
@@ -64,6 +38,32 @@ var menu = (function(){
 }
 };
 
+varbind={};
+
+bind.focus={
+loop: function(event){
+varfirstElement=helper.e(".control-menu-layout");
+varlastElement=helper.e(".menu-close-tab");
+if(event.keyCode==9&&event.shiftKey){
+if(document.activeElement===firstElement){
+lastElement.focus();
+event.preventDefault();
+}
+}elseif(event.keyCode==9){
+if(document.activeElement===lastElement){
+firstElement.focus();
+event.preventDefault();
+}
+};
+},
+add: function(){
+window.addEventListener("keydown",bind.focus.loop,false);
+},
+remove: function(){
+window.removeEventListener("keydown",bind.focus.loop,false);
+}
+};
+
 render={};
 
 render.panel=function(){
@@ -257,6 +257,7 @@ var menu = (function(){
 return{
 init: init,
 mod: mod,
+bind: bind,
 render: render,
 open: open,
 close: close,

src/js/modal.js

@@ -20,6 +20,33 @@ var modal = (function(){
 });
 };
 
+varbind={};
+
+bind.focus={
+loop: function(event){
+varallElements=helper.e(".modal").querySelectorAll("[tabindex]");
+varfirstElement=allElements[0];
+varlastElement=allElements[allElements.length-1];
+if(event.keyCode==9&&event.shiftKey){
+if(document.activeElement===firstElement){
+lastElement.focus();
+event.preventDefault();
+}
+}elseif(event.keyCode==9){
+if(document.activeElement===lastElement){
+firstElement.focus();
+event.preventDefault();
+}
+};
+},
+add: function(){
+window.addEventListener("keydown",bind.focus.loop,false);
+},
+remove: function(){
+window.removeEventListener("keydown",bind.focus.loop,false);
+}
+};
+
 varrender={};
 
 render.toggle=function(override){
@@ -144,11 +171,13 @@ var modal = (function(){
 varopen=function(override){
 mod.open();
 render.open(override);
+bind.focus.add();
 };
 
 varclose=function(){
 mod.close();
 render.close();
+bind.focus.remove();
 };
 
 varinit=function(){
@@ -160,6 +189,7 @@ var modal = (function(){
 return{
 init: init,
 mod: mod,
+bind: bind,
 render: render,
 open: open,
 close: close