Changeset 23455

Timestamp:

2014/05/28 04:45:48 (10 years ago)

Author:

Seasoft

Message:

#2157 (Windows8対応)

• クリックでメニューの第1階層を閉じた場合に再び開く不具合を回避
• コメント・命名をマシに

寝ます

File:

• branches/version-2_13-dev/html/user_data/packages/admin/js/eccube.admin.js (modified) (6 diffs)

branches/version-2_13-dev/html/user_data/packages/admin/js/eccube.admin.js

@@ -358 +358 @@
     eccube.navi = {};
 
-    // メニューを閉じないフラグ
-    eccube.navi.not_close = false;
-
+    // クリックを無視するフラグ
+    eccube.navi.ignore_click = false;
+
+    /**
+     * 対象を指定してメニューを開く
+     */
     eccube.navi.openMenu = function($target) {
         $target
@@ -371 +374 @@
     };
 
-    eccube.navi.setNotClose = function(milliseconds) {
+    /**
+     * 全てのメニューを閉じる
+     */
+    eccube.navi.closeAllMenu = function() {
+        $("#navi")
+            .removeClass('active')
+            .find('li')
+                .removeClass('sfhover')
+    }
+
+    /**
+     * 一時的にクリックイベントを無視する
+     */
+    eccube.navi.setIgnoreClick = function(milliseconds) { // FIXME: 関数名
         if (milliseconds = null) milliseconds = 100;
-        eccube.navi.not_close = true;
-        setTimeout(function(){eccube.navi.not_close = false;}, milliseconds);
+        eccube.navi.ignore_click = true;
+        setTimeout(function(){eccube.navi.ignore_click = false;}, milliseconds);
     }
 
@@ -388 +404 @@
         $("#navi").find("div").click(function(){
             // タブレットでの二重イベント発生を回避
-            if (eccube.navi.not_close) return false;
+            if (eccube.navi.ignore_click) return false;
 
             naviClicked = true;
@@ -399 +415 @@
                 eccube.navi.openMenu(parent);
             } else {
-                parent.removeClass('sfhover');
+                if (parent.hasClass('on_level1')) {
+                    eccube.navi.closeAllMenu();
+                } else {
+                    parent.removeClass('sfhover');
+                }
             }
         });
@@ -406 +426 @@
             if ($("#navi").hasClass('active')) {
                 eccube.navi.openMenu($(this));
-                eccube.navi.setNotClose();
+                eccube.navi.setIgnoreClick();
             }
         });
@@ -412 +432 @@
         $(document).click(function(){
             if (!naviClicked) {
-                $("#navi")
-                    .removeClass('active')
-                    .find('li')
-                        .removeClass('sfhover')
+                eccube.navi.closeAllMenu();
             } else {
                 naviClicked = false;