cferdinandi / drop

A small CSS component that turns browser-native <details> elements into dropdown menus.
MIT License
77 stars 12 forks source link

Drop Jumbo (with Astro Navbar). Added sub-links not working in ProcessWire CMS #40

Closed chrisave closed 9 years ago

chrisave commented 9 years ago

I'm trying to use Drop Jumbo (with Astro Navbar) with ProcessWire for a project.

The static basis is:

So, I've replaced divs and texts (only) with an ul, lists and anchors (if compared to the demo).

Perhaps because of this (an active class being applied to a second-level anchor(?)) or something else, when I click on a sub-link in the drop-down menu, it doesn't load the page targeted by the anchor/link (but the URL is fine). There is the following message in Firebug's Console(?): "TypeError: i is null

...e.classList.toggle(o.toggleActiveClass),i.classList.toggle(o.contentActiveClass)..."

The fallback URL is the first sub-link URL.

cferdinandi commented 9 years ago

So this is a bug in 9.2.0. Please upgrade to 9.2.1 and the problem will go away. Sorry!

https://github.com/cferdinandi/drop/releases/tag/v9.2.1

chrisave commented 9 years ago

Thank you for fixing it. Have a nice day :)!