Question

主菜单的点击代码用于更改类别以及所有但不在子菜单...当点击食物或鞋子等子菜单项时它不会触发警报命令...实际上悬停对于子菜单非常适用但不是活跃的

HTML

<aside>
    <div class=sidebar>
        <ul class=nav>
            <li class="home active" data-class="page_home">HOME</li>
            <li class="products" data-class="page_products">BYE</li>
            <li class="ART" data-class="page_art">ART</li>
        </ul>
    </div>
</aside>
<article>
    <div class="page_home show">
        <div class="myhome">
             <h1>Home</h1>

            <p>this is the index page</p>
        </div>
    </div>
    <div class="page_products">
        <ul class="productcat">
            <li class="food">FOOD</li>
            <li class="shoes">SHOES</li>
            <li class="cloths">CLOTHS</li>
        </ul>
    </div>
    <div class="page_art">
        <div class="myart">
             <h1>ART</h1>

            <p>this is art page</p>
        </div>
    </div>
</article>

JS

$(".page_products ul.productcat li").on('click', function (e) {
    var $this = $(this),
        // Class of the one to be shown
        subclazz = $this.data("class");
    alert("button clicked");

    $this.addClass('active').siblings().removeClass('active');
}

$("ul.nav li").on('click', function (e) {
    var $this = $(this),
        // Class of the one to be shown
        clazz = $this.data("class");

    $this.addClass('active').siblings().removeClass('active');

    $('article > div').removeClass('show');
    $('.' + clazz).addClass('show');

});

CSS

body {
    width:1020px;
    margin:0 auto;
    position:absolute;
}
aside {
    float:left;
    display:block;
    width:20%;
}
article {
    float:right;
    display:block;
    margin-top:0;
    width:75%;
}
.page_home, .page_products, .page_art{
    visibility:hidden;
}
.show {
    visibility: visible;
}
.sidebar {
    margin-left:10px;
}
.nav {
    display:block;
    margin-top:60px;
    list-style-type:none;
}
ul {
    list-style-type:none;
    color:#000;
}
.nav > li {
    background: #666 no-repeat;
    width:150px;
    height:65px;
    color:#000;
}
.nav >li:hover {
    background:#06F no-repeat;
}
.nav > li.active {
    background:#06F no-repeat;
}


.productcat > li {
    background: #666 no-repeat;
    width:150px;
    height:65px;
    color:#fff;   
    display:inline; 
    padding:10px;
}


.productcat>li:hover {
background:#06F no-repeat;

}

.productcat > li.active {
background:#06F no-repeat;
}

Answer 1

尝试使用

$(".page_products ul.productcat li").on('click', function (e) {
    var $this = $(this);
        // Class of the one to be shown
        subclazz = $this.attr("class");
        alert("button clicked");

    $this.addClass('active').siblings().removeClass('active');
}

我改变了

subclazz = $this.data("class");

到

subclazz = $this.attr("class");

和

var $this = $(this),

到

var $this = $(this);

Answer 2

您的代码有两个问题..

您忘记了第一个事件处理程序的结束括号

$(".page_products ul.productcat li").on('click', function (e) {
    var $this = $(this),
        // Class of the one to be shown
        subclazz = $this.data("class");
    alert("button clicked");

    $this.addClass('active').siblings().removeClass('active');
});
 ^---- Missing closing braces here

其次，您使用data方法访问自定义方法

  subclazz = $this.data("class");

但您使用的HTML是以这种形式

<li class="food">FOOD</li>

因此，如果您希望data方法有效，那么您的HTMl结构应该是

  <li data-class="food">FOOD</li>

此外，您没有在代码中的任何位置使用此变量，这是不必要的。我很惊讶你说你的第二个事件处理程序在你的语法错误后工作。

在这种情况下，Hover在这里工作，因为使用CSS而不是javascript应用样式，并且在应用样式时javascript中的错误没有任何效果。

您始终可以通过在开发人员工具中检查控制台来检查js中的错误。

<强> Check Fiddle

子菜单点击事件不工作/解雇

2 个答案: