ComputerWolf / SlickNav

Responsive Mobile Menu Plugin for jQuery
MIT License
941 stars 314 forks source link

JQuery version 3.2.1 #162

Closed RudyMas closed 7 years ago

RudyMas commented 7 years ago

I'm trying to get SlickNav to work on my website, but whatever I try, it doesn't want to work. It also made me wonder if your code is compatible with JQuery version 3.2.1? If so...

My site is online at https://rmfoto.be/

Any help is appreciated.

Best regards, Rudy Mas

RudyMas commented 7 years ago

Typical! Just after I asked this question, I found the mistake.

You mentioned to use:

.slicknav_menu { display:none; }

@media screen and (max-width: 40em) { / #menu is the original menu / .js #menu { display:none; }

.js .slicknav_menu {
    display:block;
}

}

But it should to be:

.slicknav_menu { display:none; }

@media screen and (max-width: 40em) { / #menu is the original menu /

menu {

    display:none;
}

.slicknav_menu {
    display:block;
}

}

The .js isn't needed. :)

However, I noticed a strange behavior. When I click on the year 2012, it only opens or closes the submenu, while I had hoped that double clicking on it would open the link underneath it.

Could you look at jquery-doubletaptogo to implement something like that. It would be greatly appreciated.

Nevertheless, thank you very much for SlickNav, because it looks much better than anything I tried to write myself. :)

Best regards, Rudy Mas