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

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

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

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

Q&A

解決済

2回答

760閲覧

jsでHTMLのdata属性を扱いたい

Yariii

総合スコア61

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

0グッド

0クリップ

投稿2020/11/15 13:38

wordpressです。
htmlで以下のようなブロックがあり、所々にdata-nameがあり、画像のurlが値に入っています。
そしてスクロールした時に、data-nameがあるところにスクロールが行き着いたら、そのdata-nameの値(画像url)を背景写真として切り替える、という風な挙動を書きたいです。

その上でjsでループ処理をしているのですが、ループ内でのdata-nameの扱い方がわからず困っています。

html

1///背景切り替え 2<section class="History_timeline_point trigger" data-name="<?php echo esc_url(get_template_directory_uri()); ?>/img/history0.jpg"> 3 <div class="year">アイテム1</div> 4</section> 5 6<section class="History_timeline_point"> 7 <div class="year">アイテム2</div> 8</section> 9 10///背景切り替え 11<section class="History_timeline_point trigger" data-name="<?php echo esc_url(get_template_directory_uri()); ?>/img/history0.jpg"> 12 <div class="year">アイテム3</div> 13</section> 14 15<section class="History_timeline_point"> 16 <div class="year">アイテム4</div> 17</section> 18

js

1 2 let trigger = $('.trigger'); 3 let backTrigger = new Array; 4 let backImg = new Array; 5 6 //背景セット 7 trigger.each(function(i){ 8 //それぞれトリガーの高さを入れる 9 backTrigger[i] = $(this).offset().top; 10  //それぞれのdata-name(背景画像)をセット 11 backImg[i] = $(this).dataset.name; 12 }); 13

上記jsコードの、eachを回すところで、
backImgにdata属性をそれぞれ配列にセットしたいのですが、
backImg[i] = $(this).dataset.name;
の部分ががエラーになります。

TypeError: undefined is not an object (evaluating '$(this).dataset.name')

ループ内ではどのようにdata-nameの値を指定できるのでしょうか?
ご教授よろしくお願い申し上げます。

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

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

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

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

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

guest

回答2

0

ベストアンサー

$を外して、this.dataset.nameとするのが手っ取り早いです。

投稿2020/11/15 13:41

maisumakun

総合スコア145208

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

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

0

jQueryですよね?

.data('name')

投稿2020/11/15 14:06

yambejp

総合スコア115010

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

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

maisumakun

2020/11/15 14:12

.dataからdata-xxx属性を参照すると、最初にデータを取った時点でjQuery内にキャッシュされてしまうので、jQuery外からdata-xxx属性を動かすような場面で使うには向かないことが考えられます。
Yariii

2020/11/15 15:14

ありがとうございます。 $()をつけるとjQueryの中に格納されたから?なんですね。勉強になりました。
yambejp

2020/11/15 15:19

ああ、失礼しました jQueryのdataは必ずしもカスタムデータと合致しませんね .prop('dataset')['name'] とか .attr('data-name'') で参照するべきでした
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問