菜单图标切换问题 JQuery

Question

我获得了一个切换控件，该控件在单击时在菜单图标和X 之间进行动画处理。它还提供了一个覆盖导航。

问题是：当您单击覆盖导航上的任何链接时，它不会将X 更改/切换回默认菜单图标。

---

在下面的代码 sn-p 中，我添加了 $('.cd-menu-icon').toggleClass('is-clicked'); $('.cd-header').toggleClass('menu-is-open');（在第 33、34 行）来初始设置控件的切换状态。

jQuery(document).ready(function($){
    //if you change this breakpoint in the style.css file (or _layout.scss if you use SASS), don't forget to update this value as well
    var MQL = 1170;

    //primary navigation slide-in effect
    if($(window).width() > MQL) {
        var headerHeight = $('.cd-header').height();
        $(window).on('scroll',
        {
            previousTop: 0
        }, function () {
            var currentTop = $(window).scrollTop();
            //check if user is scrolling up
            if (currentTop < this.previousTop ) {
                //if scrolling up...
                if (currentTop > 0 && $('.cd-header').hasClass('is-fixed')) {
                    $('.cd-header').addClass('is-visible');
                } else {
                    $('.cd-header').removeClass('is-visible is-fixed');
                }
            } else {
                //if scrolling down...
                $('.cd-header').removeClass('is-visible');
                if( currentTop > headerHeight && !$('.cd-header').hasClass('is-fixed')) $('.cd-header').addClass('is-fixed');
            }
            this.previousTop = currentTop;
        });
    }

    //open/close primary navigation
    $('.cd-primary-nav-trigger').on('click', function(){
        $('.cd-menu-icon').toggleClass('is-clicked');
        $('.cd-header').toggleClass('menu-is-open');

        //in firefox transitions break when parent overflow is changed, so we need to wait for the end of the trasition to give the body an overflow hidden
        if( $('.cd-primary-nav').hasClass('is-visible') ) {
            $('.cd-primary-nav').removeClass('is-visible').one('webkitTransitionEnd otransitionend oTransitionEnd msTransitionEnd transitionend',function(){
                $('body').removeClass('overflow-hidden');
            });
        } else {
            $('.cd-primary-nav').addClass('is-visible').one('webkitTransitionEnd otransitionend oTransitionEnd msTransitionEnd transitionend',function(){
                $('body').addClass('overflow-hidden');
            });
        }
    });
});

希望有人能提供帮助。

---

重现：（使用 JsFiddle Example）

1. 打开菜单（点击“汉堡”图标框区域）
2. 点击覆盖上的任何链接，当它出现时。 （即“1”、“2”、“3”、“4”）
3. 结果：覆盖关闭，但菜单图标没有切换回来。

Answer 1

我认为您已经在 HTML 部分的 <script> 标签 （第 253 行） 中实现了 1, 2, 3, 4 链接上的自定义点击事件。

只需将 <script> 标记替换为以下内容：

<script>
  $('.cd-primary-nav a').on('click', function(){
  $('.cd-menu-icon').removeClass('is-clicked');
  $('.cd-header').removeClass('menu-is-open');
  $('.cd-primary-nav').removeClass('is-visible').one('webkitTransitionEnd otransitionend oTransitionEnd msTransitionEnd transitionend',function(){
    $('body').removeClass('overflow-hidden');
  });
  });
</script>

JSFiddle：https://jsfiddle.net/c9dj010v/

Answer 2

一个快速的解决方案是添加一个新的点击监听器来监听链接上的点击

$('.nav-overlay').click(function() {
    $('.cd-menu-icon').toggleClass('is-clicked');
});

在您原来的小提琴中，汉堡图标仅在直接单击时才会更改，因为单击侦听器仅附加到该元素

Modified Fiddle