タブをクリックすると上部にスクロールされてしまう

Question

はじめまして。jQuery初心者です。
http://www.aiship.jp/knowhow/archives/28160

上記サイトのタグを拝借し、いい感じのところまでいっております。

ただ、タブをクリックする度に徐々に画面の上部にスクロールしてしまいます。
タブをクリックすればするほど少しずつ上部に移動し、
最終的には一番上に…

何かいい解決方法教えいただけませんでしょうか？<(_ _)>

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
(追加)
すみません。少し原因が分かってきました。

common.jsファイル

// smooth scroll
$(function(){
	$('a[href^=#]').not('#header a,.tab li a,').click(function(){
		  var speed = 500;
		  var href= $(this).attr("href");
		  var target = $(href == "#" || href == "" ? 'html' : href);
		  var position = target.offset().top - 100;
		  $("html, body").animate({scrollTop:position}, speed, "swing");
		  return false;
	});
});

という記載があり、どうやらこれが影響しているようです。
$('a[href^=#]').not('#header a,.tab li a,').click(function(){
を、
$('a[href^=#]').not('#header a,.tab li a,#tabindex li a').click(function(){
と書き換えれば、問題は解決するのですが、
このcommon.jsファイル事態を触ることが禁止されています。

出来れば、test.jsファイル(以下、記載)内で解決したく、

$(function(){
    $('#tabcontents div[id != "tab01"]').hide();
    $("a").click(function(){
        $("#tabcontents div").hide();
        $($(this).attr("href")).show();
        $(".current").removeClass("current");
        $(this).addClass("current");
        return false;
    });
 });

上記記載内で解決できませんでしょうか？

htmlも念のため記載しておきます

<section id="tabindex">
   <ul>
    <li><a href="#tab01" class="current">color</a></li>
    <li><a href="#tab02" class="">size</a></li>
    <li><a href="#tab03" class="">Material</a></li>
    <li><a href="#tab04" class="">Country</a></li>
    <li><a href="#tab05" class="">aa</a></li>
  </ul>
  
  <div id="tabcontents">
    <div id="tab01">
      <p>
      <strong>Blue/Red/White</strong></p>
      <p></p>
    </div>
    <div id="tab02">
      <p><strong>36・38・40</strong></p>
      <p></p>
    </div>
    <div id="tab03">
      <p><strong>cotton 100%</strong></p>
      <p></p>
    </div>
    <div id="tab04">
      <p><strong>Japan</strong></p>
      <p></p>
    </div>
    <div id="tab05">
      <p><strong>aaa</strong></p>
      <p></p>
    </div>
  </div>
    
    </section>

ちなみに、<ul class="tab">としたら、タブのスクロールは解決しましたが、
タブ下の表示部分が崩れました( ﾉД`)ｼｸｼｸ…

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝追加＝＝＝＝＝＝＝

a要素でクリックせずに～を試しました
解決できましたー(^^)/

$(function(){
    $('#tabcontents div[id != "tab01"]').hide();
	$( 'span[data-href]' ).on( 'click', function() {
        $("#tabcontents div").hide();
	$( $( this ).data( 'href') ).show();
        $(".current").removeClass("current");
        $(this).addClass("current");
        return false;
    });
});

<section id="tabindex">
  
   <ul>
    <li><span data-href="#tab01" class="current">color</span></li>
    <li><span data-href="#tab02" class="">size</span></li>
    <li><span data-href="#tab03" class="">Material</span></li>
    <li><span data-href="#tab04" class="">Country</span></li>
    <li><span data-href="#tab05" class="">aa</span></li>
  </ul>
  
  <div id="tabcontents">
    <div id="tab01">
      <p>
      <strong>Blue/Red/White</strong></p>
      <p></p>
    </div>
    <div id="tab02">
      <p><strong>36・38・40</strong></p>
      <p></p>
    </div>
    <div id="tab03">
      <p><strong>cotton 100%</strong></p>
      <p></p>
    </div>
    <div id="tab04">
      <p><strong>Japan</strong></p>
      <p></p>
    </div>
    <div id="tab05">
      <p><strong>aaa</strong></p>
      <p></p>
    </div>
  </div>
    
    </section>

Comments

[kei344]

JQとはjQueryのことでしょうか。また、書かれている状況が再現するコード（HTML/CSS/JavaScriptなど）を提示されたほうが回答を得やすいと思います。

[kei344]

質問文のコードはそれぞれコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

Answer 1

【Check! jQuery ですでに登録されているイベントを取得し、割込ませたりするメモ - Qiita】
http://qiita.com/dz_/items/e5da8ed692720a69126c

上記記事のコードを少し改変し、イベントを付け直すことが出来ることを確認しました。

$( function() {
  // 元のクリックイベントを設定する
  $( '#submit' ).on( 'click', function() {
    console.log( 'Message is ' + $( this ).text() );
  });

  // クリックイベントに設定されたハンドラを取得する
  var events = $._data( $( '#submit' ).get( 0 ), 'events' );
  var originalHandler = events.click[ 0 ].handler;

  // 元のクリックイベントを外す
  $( '#submit' ).off( 'click', originalHandler );

  // セレクタを変えて、クリックイベントに、元のハンドラを設定する
  $( '#submit, #message' ).on( 'click', originalHandler );
} );
```**動くサンプル：**[https://jsfiddle.net/f2dtneh6/1/](https://jsfiddle.net/f2dtneh6/1/)

---

**追記：**

a要素でクリックせずに、span要素などに書き換えてしまえば他の関数のことを考える手間も無いでしょう。

```HTML
<!-- 
<li><a href="#tab01" class="current">color</a></li>
↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓ -->
<li><span data-href="#tab01" class="current">color</span></li>

// $( 'a' ).click( function(){
// ↓
   $( 'span[data-href]' ).on( 'click', function() {

// $( $( this ).attr( 'href') ).show();
// ↓
   $( $( this ).data( 'href' ) ).show(); // $( this ).data( 'data-href' ) から修正
```**動くサンプル：**[https://jsfiddle.net/u0g0p7k5/](https://jsfiddle.net/u0g0p7k5/)

Comments

[kita]

ありがとうございます。。。ですが、ほとんどよくわかりません(-_-;)面目ないです。
上記に記載いただいたコードをtest.jsにそのまま記載すれば、解決でしょうか？
試みたのですが、変化なしで…編集が必要でしょうか？？

[kei344]

回答したのは実証用コードです。ご自身の環境に合わせて編集することが必要になります。

[kita]

すみません。超初心者なので、どの辺をどう編集すべきかもご教授いただけると有り難いです。。。
ご面倒おかけして申し訳ありません<(_ _)>

[kei344]

// クリックイベントに設定されたハンドラを取得する
より前は実証コード用のものなので無視してしまってください。

$( '#submit' ) のクリックイベントに割り当てられた関数を originalHandler に入れています。

何がどうわからないかわからないため、これ以上なんとも回答できません。正直この解決方法は「初心者向き」ではないので、不適切だったかもしれません。

[kita]

色々ありがとうございました。なんとか上記情報で頑張ってみます。。
私自身、このサイトを利用するのも初めて、さらに仰る通り「何がどうわからないかわからない」状況です。
こんな状態で大変失礼いたしました。
度重なるご協力、ありがとうございました<(_ _)>

[kei344]

あ、追記の分、$( this ).data( 'href')にしてください。回答も修正しておきます。

[kita]

あっ！！上手くいきました！！すごい！かんどーー(=ﾟωﾟ)ﾉ
本当に長々とお付き合いいただきありがとうございました！
本当に感謝・感激です！ありがとーーー！！

Answer 2

恐らくaタグでタブが構成されていて、href="#"になっているのではありませんか？
であればhrefの動きをキャンセルしないとトップに飛びます。URLに＃が付いていると思います。

$('a').on('click', function()){
 // タブ処理
});

みたいになっている部分があると思いますので

$('a').on('click', function()){
 // タブ処理
 return false;
});

にすればhrefの動きをキャンセル出来ます。

Comments

[kita]

return false;
は既に入っています。。。

[takepieee]

test.jsにも同じ発火条件（$('a[href^=#]').not('#header a,.tab li a,#tabindex li a').click(function(){ ）にしてみてはいかがでしょうか？

[kei344]

To: takepieeeさん
同じ発火条件にしても先に登録された関数からの発火になると思います。

[kita]

$("a").click(function(){の箇所を上記に変更で合ってますか？
試みましたが、上手くいきませんでした。。。