why alert not working in js and index return -1?

Question

HTML Code:

<ul id="menu-controls">
  <li>
    <a target="_self" onclick="return displaySubMenu(0);" id="menu-controls-0" href="#" class="">
      <span>
        Home
      </span>
    </a>
  </li>
  <li>
    <a target="_self" onclick="return displaySubMenu(1);" id="menu-controls-1" href="#" class="">
      <span>
        Oleg Test
      </span>
    </a>
  </li>
  <li>
    <a target="_self" onclick="return displaySubMenu(2);" id="menu-controls-2" href="#" class="act">
      <span>
        Products
      </span>
    </a>
  </li> 
</ul>

Jquery Code :

function displaySubMenu(menuid) {

    var myActive = jQuery("a.act").parent().index();
    var myClicked = jQuery("#menu-controls-"+menuid).parent().index();

    alert(myActive+"==="+myClicked);

    if (myActive < myClicked) {
        alert("Previous active was left");
    } else if (myActive > myClicked) {
        alert("Previous active was right");
    } else if (myActive == myClicked) {
        alert("Previous active is this clicked one");
    }

    jQuery("a.act").removeClass("act");
    jQuery("#menu-controls-"+menuid).addClass("act");
    return false;
}

Seems to be working fine: [**Updated Fiddle**](http://jsfiddle.net/xXvYK/2/) — palaѕн, Oct 08 '13 at 10:27
http://jsfiddle.net/xXvYK/3/ select `no-wrap in body` option while calling javascript function :) in fiddle options — Dhaval Marthak, Oct 08 '13 at 10:27

score 0 · Accepted Answer · answered Oct 08 '13 at 10:29

0

Your fiddle works. Just select the No Wrap - in (head) option on the left side under Frameworks & Extensions section.

answered Oct 08 '13 at 10:29

Gurminder Singh

1,755
16
19

why this No Wrap - in (head) ? – Shakti Patel Oct 08 '13 at 10:31
1

"That simply means place your JavaScript between the tags". See [REFERENCE](http://stackoverflow.com/questions/14036969/nowrap-condition-in-html-head-and-body). – Gurminder Singh Oct 08 '13 at 10:34
And you are getting index 2 because the elements which you are selecting are placed at 3rd position i.e. index number 2. – Gurminder Singh Oct 08 '13 at 10:36

why alert not working in js and index return -1?

1 Answers1