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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Chrome

Google Chromeは携帯、テレビ、デスクトップなどの様々なプラットフォームで利用できるウェブブラウザです。Googleが開発したもので、Blink (レンダリングエンジン) とアプリケーションフレームワークを使用しています。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

HTML

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

CSS

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

Q&A

解決済

1回答

5512閲覧

デベロッパーツールが表示されたときのみ動くJSを表示されてなくても動くようにしたいです

Raiden

総合スコア22

Chrome

Google Chromeは携帯、テレビ、デスクトップなどの様々なプラットフォームで利用できるウェブブラウザです。Googleが開発したもので、Blink (レンダリングエンジン) とアプリケーションフレームワークを使用しています。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

HTML

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

CSS

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

0グッド

0クリップ

投稿2019/03/04 17:59

前提・実現したいこと

スクロールに応じて上部に固定した水平メニューのスタイルがかわる機能をJSで作りました。
menu
works
about us
contact
のような順でスクロールするとコーナーが進む、いわゆるランディングページのような作りです。
スクロールをして該当のコーナーが表示されると上部メニューにCSSスタイルが適用されるようにしたいです。

発生している問題

他のJSファイルは動くのですが、このJSファイルだけ動きません。
原因を確かめようとChromeでデベロッパーツールを開くと、なんとこの機能が動き出します。
書いたとおりにスクロールに応じて該当のメニューのスタイルがかわります。
該当のソースコードにエラーは出ません。
ブレークポイントを置いてステップ実行させても、特に問題は見当たりません。
デベロッパーツールを閉じた後も動きます。
しかし、更新するとまた同じ問題がおきて動きません。

該当のソースコード

index.html

<!DOCTYPE html> <nav id="page-top" class="navbar navbar-inverse navbar-fixed-top header-change"> <div class="container-fluid"> <!-- ヘッダー部分 ================ --> <div class="navbar-header"> <a class="navbar-brand" href="#top"> <img class="navLogo" src="img/logo.png"> </a> <button type="button" class="navbar-toggle collapsed" data-toggle="collapse" data-target="#nav_target"> <span class="icon-bar"></span> <span class="icon-bar"></span> <span class="icon-bar"></span> </button> </div> <div class="collapse navbar-collapse" id="nav_target"> <ul class="nav navbar-nav"> <li id="list02" class="menu-list"><a href="#news">NEWS</a></li> <li id="list03" class="menu-list"><a href="#workstop">WORKS</a></li> <li id="list04" class="menu-list"><a href="#aboutus">ABOUT</a></li> <li id="list05" class="menu-list"><a href="#contact">CONTACT</a></li> </ul> <ul class="nav navbar-nav navbar-right"> <li><a href="#"></i></a></li> <li><a href="#" target="_blank"><i class="fab fa-youtube"></i></a></li> </ul> </div> </div> </nav>

style.scss

.current { color: indigo; font-weight: 900; }

naveffect.js

$(function() { var topBtn = $('#page-top'); $("#page-top").addClass("header-change"); $(window).scroll(function() { $(window).on('load resize', function() { var pos01 = 0; var pos02 = Math.round($("#news").offset().top); var pos03 = Math.round($("#workstop").offset().top); var pos04 = Math.round($("#aboutus").offset().top); var pos05 = Math.round($("#contact").offset().top); $(window).on('load resize scroll', function() { var posScroll = $(window).scrollTop() + (window.innerHeight / 2); if (posScroll <= pos01) { $('.menu-list').css('background-color', 'transparent'); $(".nav li").removeClass('current'); } else if (pos02 <= posScroll && posScroll < pos03) { $('.menu-list').css('background-color', 'transparent'); $(".nav li").removeClass('current'); $("#list02").addClass('current'); } else if (pos03 <= posScroll && posScroll < pos04) { $('.menu-list').css('background-color', 'transparent'); $(".nav li").removeClass('current'); $("#list03").addClass('current'); } else if (pos04 <= posScroll && posScroll < pos05) { $('.menu-list').css('background-color', 'transparent'); $(".nav li").removeClass('current'); $("#list04").addClass('current'); } else if (pos05 <= posScroll) { $('.menu-list').css('background-color', 'transparent'); $(".nav li").removeClass('current'); $("#list05").addClass('current'); } }); }); }); });

試したこと

Firefoxでも同じ事が起こりました。

上にもかきましたが、デベロッパーツールを開いてステップ実行をして何かエラーが出ないか探りましたが出なかったです。

<script src="naveffect.js"></script>

htmlでは上記を読み込んでいるので、headerなどに位置をかえてみたりしましたが何もかわりませんでした。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

イベントの中でイベントを登録しない。するならきちんとイベントを外しましょう。

スクロールイベントはスクロール中何度も呼ばれます。
そのたびに「'load resize'」イベントを追加している状態です。
(デベロッパーツールを開かなくてもブラウザのウィンドウサイズを変更したら動くと思われます)

js

1 $(window).scroll(function() { 2 $(window).on('load resize', function() { 3 $(window).on('load resize scroll', function() { 4 }); 5 }); 6 }); 7```8```js 9 // 共通で使う変数はこの部分で宣言 10 $(window).on('load resize', function() { 11 // 初期化コード 12 }); 13 $(window).scroll(function() { 14 // スクロール時のコード 15 });

投稿2019/03/04 18:13

kei344

総合スコア69407

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Raiden

2019/03/04 18:25

ご指摘の通りでした。 入れ子になっていてパフォーマンスも下がっていたようで、修正したらだいぶ軽くなりました。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問