Side navigation links don't incur any actions, since the click function has a preventDefault() applied. To make the side navigation work, we need to remove e.preventDefault() in STARTERKIT/js/script.js, line number 42.

CommentFileSizeAuthor
#6 2951924-6-fix-for-enable-side-navigation-links.patch409 bytesBettina Händel
#4 sidenav.jpg21.42 KBdenes.szabo

Comments

igel created an issue. See original summary.

denes.szabo’s picture

denes.szabo commented

The line 42. is about closing the sidenav.

What browser did you use? It worked to me in Firefox, Chrome. I will check it again soon.

igel’s picture

igel commented

I can tell, yet removing this line fixes the issue for me. I'm using Opera 51.0 (Mac).

denes.szabo’s picture

denes.szabo commented
StatusFileSize
new sidenav.jpg21.42 KB

@igel - The event.preventDefault is for: For example, clicked anchors will not take the browser to a new URL. http://api.jquery.com/event.preventDefault/

This is we need there, handling the click event if the close link clicked - remove body class then. No need this event bubble up.

Maybe, in your HMTM the sidenav close overflows the sidenav menu links?

This is how it should be display:

Sidenav

Would you mind to check your version?

denes.szabo’s picture

denes.szabo commented
Status: Active » Postponed (maintainer needs more info)

Just postponed till new info arrives.

Bettina Händel’s picture

Bettina Händel
Location Munich
commented
StatusFileSize
new 2951924-6-fix-for-enable-side-navigation-links.patch409 bytes

With click on side navigation menu e.stopPropagation() stopps now calling $('.side-nav, .sidenav-close .button-close').click and links on side navigation are enabled :)