質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • HTML

    9000questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    6721questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • CSS

    5808questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,663

kita

score 4

はじめまして。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>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kei344

    2017/02/08 14:27

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

    キャンセル

  • kei344

    2017/02/08 14:40

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

    キャンセル

回答 2

checkベストアンサー

+3

【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/


追記:

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

<!-- 
<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/

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/08 16:00

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

    キャンセル

  • 2017/02/08 16:10

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

    キャンセル

  • 2017/02/08 16:18

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

    キャンセル

  • 2017/02/08 16:35

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

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

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

    キャンセル

  • 2017/02/08 16:40

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

    キャンセル

  • 2017/02/08 17:18

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

    キャンセル

  • 2017/02/08 17:26

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

    キャンセル

+1

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

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


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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/08 14:31

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

    キャンセル

  • 2017/02/08 16:01

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

    キャンセル

  • 2017/02/08 16:11

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

    キャンセル

  • 2017/02/08 16:16

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • HTML

    9000questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    6721questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • CSS

    5808questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。