Jane
Jane

Reputation: 987

Accordion menu, expand on first click, go to URL if sub-menu is expanded

I have a script which expands a menu, but what I want it to do is expand on the first click and then once the sub-menu is expanded, I want the expanding link to go to actually go to the URL specified if it's clicked again. Here is the code I have so far:

jQuery

$( document ).ready( function( ) {
            $( '.menu li' ).each( function() {
                if( $( this ).children( 'ul' ).length > 0 ) {
                            $( this ).addClass( 'parent' );     
                    }
            });

            $( '.menu li.parent > a' ).click( function( ) {
                    event.preventDefault();
                    $( this ).parent().toggleClass( 'active' );
                    $( this ).parent().children( 'ul' ).slideDown( 'fast' )

                    window.location.href = $(this).attr('href');

            });

    });

HTML

<div class="menu">
    <ul>
        <li><a href="http://www.google.com">Menu item 1</a>
            <ul>
                <li><a>Menu item 1.1</a></li>
                <li><a>Menu item 1.2</a>
                    <ul>
                        <li><a>Menu item 1.2.1</a></li>
                        <li><a>Menu item 1.2.2</a></li>
                        <li><a>Menu item 1.2.3</a></li>
                    </ul>
                </li>
                <li><a>Menu item 1.3</a></li>
            </ul>
        </li>
    </ul>
</div>

CSS

a
    {
        cursor: pointer;
    }

.menu ul 
    {
        list-style: none outside none;
    }

.menu li a 
    {
        line-height: 25px;
    }

.menu > ul > li > a 
    {
        color: #3B4C56;
        display: block;
        font-weight: normal;
        position: relative;
        text-decoration: none;
    }

.menu li.parent > a 
    {
        padding: 0 0 0 28px;
    }

.menu li.parent > a:before 
    {
        background-image: url("../images/plus_minus_icons.png");
        background-position: 25px center;
         content: ""; 
        display: block;
        height: 21px;
        left: 0;
        position: absolute;
        top: 2px;
        vertical-align: middle;
        width: 23px;
    }

.menu ul li.active > a:before 
    {
        background-position: 0 center;
    }

.menu ul li ul 
    {
        border-left: 1px solid #D9DADB;
        display: none;
        margin: 0 0 0 12px;
        overflow: hidden;
        padding: 0 0 0 25px;
    }

.menu ul li ul li 
    {
        position: relative;
    }

.menu ul li ul li:before 
    {
        border-bottom: 1px dashed #E2E2E3;
        content: "";
        left: -20px;
        position: absolute;
        top: 12px;
        width: 15px;
    }

JSFIDDLE

Upvotes: 2

Views: 1329

Answers (2)

dm4web
dm4web

Reputation: 4652

UPDATE at the suggestion of @dippas

http://jsfiddle.net/g3o64ve7/3/

use is and hidden

$( document ).ready( function( ) {
        $( '.menu li' ).each( function() {
                if( $( this ).children( 'ul' ).length > 0 ) {
                        $( this ).addClass( 'parent' );     
                }
        });

        $( '.menu li.parent > a' ).click( function(event) {

            if($(this).next('ul').is(':hidden'))
            {
                event.preventDefault();            

                $( this ).parent().toggleClass( 'active' );
                $( this ).parent().children( 'ul' ).slideDown( 'fast' );
            }
        });

});

Upvotes: 3

Madhu
Madhu

Reputation: 132

add event.preventDefault(); inside the click event. JSFIDDLE

$( '.menu li.parent > a' ).click(
    function(event) { 
        event.preventDefault();
        $( this ).parent().toggleClass( 'active' );
        $( this ).parent().children( 'ul' ).slideDown( 'fast' );
    }
);

Upvotes: 1

Related Questions