Code Changes per code review comment

- Changed fix from addressing the browser specific behavior and instead
  addressed the parent issue of NavKey events being relayed when they are not
  really NavKey events. This was per code review comment by @jpwhite4

html/gui/js/modules/metric_explorer/MetricExplorer.js

@@ -5674,7 +5674,12 @@ Ext.extend(XDMoD.Module.MetricExplorer, XDMoD.PortalModule, {
                     return true;
                 }
 
-                return h.call(this.scope || this, e, this.menu);
+                // Ensure that we only call the KeyNav handlers if this is a NavKey Press.
+                if (e.isNavKeyPress()) {
+                    return h.call(this.scope || this, e, this.menu);
+                }
+
+                return true;
             };
 
             /**
@@ -5694,26 +5699,6 @@ Ext.extend(XDMoD.Module.MetricExplorer, XDMoD.PortalModule, {
                     target.value += String.fromCharCode(key);
                 }
             };
-
-            /**
-              * Firefox + Ext 3.4 === '%' being interpreted as the 'left' key.
-              * This resulted in the code after the 'if' being executed. Which,
-              * is somewhat odd behavior to say the least ( While viewing the
-              * site in firefox,  type '%' into a text box, have the whole menu
-              * close.). This override fixes that behavior
-             **/
-            this.chartOptionsMenu.keyNav.left = function (e, m) {
-                // Gecko === Firefox
-                if (Ext.isGecko === true) {
-                    return true;
-                }
-
-                m.hide();
-                if (m.parentMenu && m.parentMenu.activeItem) {
-                    m.parentMenu.activeItem.active();
-                }
-                return true;
-            };
         }
 
         this.chartStatusButton = new XDMoD.Module.MetricExplorer.StatusButton({