質問: */ /* http://meyerweb.com/eric/tools/css/reset/ v2.0 | 20110126 License: none (public domain) */ ----------以下略 ■JavaScript jquery-1.9.1.min.js---------- /*! jQuery v1.9.1 | (c) 2005, 2012 jQuery Foundation, Inc. | jquery.org/license //@ sourceMappingURL=jquery.min.map */ ----------以下略
回答: なかなか惜しい感じになっていますね。 1 ``` var tabno = $('ul.navi_wrap li#' + tabname).index(); ``` 変数tabnameにはタブ名(tabxx)ではなくボックス名(boxxx)が入っています。 よって、 ``` var tabno = $('.box_wrap div#' + tabname).index(); ``` と変更するか、タブ名(tabxx)を指定するように変更するかのどちらかを行います。 2 ``` if($(hash).length){ ``` ここに$()はいらないです。 3 動く動かないと関係ありませんが、読み込み時と、タブクリック時で、同じ処理をしています。 読み込み時のハッシュからタブを特定し、$(tab).click()でクリックイベントを飛ばしてやれば、行数を減らせます。