StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
text
Body
copied!<p>The onclick attribute is an event handler attribute is the same as what you are doing in this line of code on your script:</p> <p>$("ol.tabs li").click(</p> <p>See: <a href="http://reference.sitepoint.com/html/event-attributes/onclick" rel="nofollow">http://reference.sitepoint.com/html/event-attributes/onclick</a></p> <p>In other words you are assigning the handling of the click event to a script by the name of '#tab1' The hash tag is an 'illegal' script char.</p> <p>To accomplish what you want there are a few thinks you can do. First use a data attribute.</p> <pre><code><ol class="tabs"> <li><a data-tab-name="#tab1">Unique benefits</a></li> <li><a data-tab-name="#tab2">Insurance</a></li> <li><a data-tab-name="#tab3">Business Loan</a></li> </ol> </code></pre> <p>And them change your js line to:</p> <pre><code>var activeTab = $(this).find("a").attr("data-tab-name"); </code></pre> <p>On a related note I will change a few more things on your script. Since you are attaching the event handler to the onclick event of the list item I will add the attribute to the list item as follow.</p> <pre><code><ol class="tabs"> <li data-tab-name="#tab1"><a>Unique benefits</a></li> <li data-tab-name="#tab2"><a>Insurance</a></li> <li data-tab-name="#tab3"><a>Business Loan</a></li> </ol> </code></pre> <p>That allows me to cahnge the js line to this:</p> <pre><code>var activeTab = $(this).attr("data-tab-name"); </code></pre> <p>What should improve performance a bit but more important makes the code more readable.</p> <p>Hope this helps.</p>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload