Changeset 23453

Timestamp:

2014/05/28 03:58:09 (10 years ago)

Author:

Seasoft

Message:

#2157 (Windows8対応)

• 単純にした。
  • 当初クリックで開いたメニューと、その後開いたメニューの扱いを同等とした。
  • ポイントでの動作を全階層同様にした。
  • 一部処理をサブルーチン化した。
• .hover() を .off() で抑制しても、再び .hover() を呼ぶとイベントがスタックされる不具合を回避。

タブレットのクリック不具合は回避できていない認識。(未検証)

File:

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

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

@@ -356 +356 @@
     };
 
+    eccube.navi = {};
+
+    eccube.navi.openMenu = function($target) {
+        $target
+            // 対象を開く
+            .addClass('sfhover')
+            // 対象以外を閉じる
+            .siblings('li')
+                .removeClass('sfhover')
+                .find('li').removeClass('sfhover');
+    };
+
     // グローバルに使用できるようにする
     window.eccube = eccube;
@@ -367 +379 @@
         $("#navi").find("div").click(function(){
             naviClicked = true;
+            $("#navi").addClass('active');
 
             var parent = $(this).parent('li');
-            var level1 = $(this).parents('li.on_level1');
-
-            if (parent.hasClass('on_level1')) {
-                // クリックしたメニューが第1階層なら開閉を切り替え.
-                if (!parent.hasClass('sfhover')) {
-                    parent.addClass('clicked');
-                } else if (parent.hasClass('clicked')) {
-                    parent.removeClass('clicked');
-                }
-            } else {
-                // 第2階層以下なら第1階層を開いた状態にする.
-                level1.addClass('clicked');
-            }
-            // クリックしたメニュー以外のメニューを閉じる.
-            level1.siblings('li')
-                .removeClass('clicked')
-                .removeClass('sfhover')
-                .find('li').removeClass('sfhover');
-
-            // ナビゲーションがアクティブであれば、マウスオーバーを有効に.
-            $("#navi")
-                .on('mouseleave', function(){
-                    $(this).not(':has(li.on_level1.clicked)').find('li')
-                        .off('mouseenter')
-                        .off('mouseleave');
-                })
-                .has('li.on_level1.clicked').find('li.on_level1').hover(
-                    function(){
-                        $(this)
-                            .addClass('sfhover')
-                            .siblings('li')
-                                .removeClass('sfhover')
-                                .removeClass('clicked')
-                                .find('li').removeClass('sfhover');
-                    },
-                    function(){
-                        $(this).not('.clicked').removeClass('sfhover');
-                    }
-                );
-
-            if (parent.hasClass('clicked')) {
-                parent.addClass('sfhover');
-            } else if (parent.hasClass('sfhover')) {
-                parent
-                    .removeClass('sfhover')
-                    .find('li').removeClass('sfhover');
-            } else {
-                parent
-                    .addClass('sfhover')
-                    .siblings('li')
-                        .removeClass('sfhover')
-                        .find('li').removeClass('sfhover');
+
+            // 開閉を切り替え.
+            if (!parent.hasClass('sfhover')) {
+                eccube.navi.openMenu(parent);
+            } else {
+                parent.removeClass('sfhover');
+            }
+        });
+        // ナビゲーションがアクティブであれば、マウスオーバーを有効に.
+        $("#navi").find('li').hover(function(){
+            if ($("#navi").hasClass('active')) {
+                eccube.navi.openMenu($(this));
             }
         });
@@ -426 +399 @@
         $(document).click(function(){
             if (!naviClicked) {
-                $("#navi").find('li')
-                    .off('mouseenter')
-                    .off('mouseleave')
-                    .removeClass('sfhover')
-                    .removeClass('clicked');
+                $("#navi")
+                    .removeClass('active')
+                    .find('li')
+                        .removeClass('sfhover')
             } else {
                 naviClicked = false;