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

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

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

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

Q&A

1回答

181閲覧

初心者(自分)が書いたjQueryのソースをきれいにしたい。

退会済みユーザー

退会済みユーザー

総合スコア0

jQuery

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

0グッド

0クリップ

投稿2018/11/12 11:46

アニメーションGIFをWEBページで表示させる際に動画埋め込み再生風のデザインで実装したいと考えています。
静止画の上に配置した、なんちゃってPNG再生ボタンを押すとYouTubeのようにアニメーションGIFが再生し、再生終了後は元に戻ります。4秒ほどしかないので停止ボタンはありません。

一応見た目は実現したのですが、jsのソースがたぶん汚いです。
cssとHTMLはなるべく追加変更せずにjsのみでもう少しスマートに書けそうなところが
ありましたらご指南お願いいたします。

このような順番になっています。
0. 静止画(01.jpg)を表示
0. 静止画の上下左右中央に再生ボタン(#btnImg、03.png)が配置されています。
0. 03.pngのボタンはマウスオーバーで透過状態に変化があったりカーソルが指アイコンになったりします。
0. 再生ボタンをクリックするとアニメーションGIF(02.gif)が再生されます。
0. アニメーションGIF再生中はボタン(03.png)が消えます。
0. 再生ボタンをクリックしてから指定時間(5秒)が過ぎると
0. 1の静止画にボタンが乗っている状態に戻ります

<!DOCTYPE html> <html lang="ja"> <head> <title></title> <link href="./base.css" rel="stylesheet"> <script src="./jquery-3.2.1.min.js"></script> <script src="./test.js"></script> </head> <body> <div class="relative"> <img id="changeImg" src="01.jpg"> <img id="btnImg" class="absolute" src="03.png"> </div> </body> </html>
.relative { position: relative; width: 600px; } .absolute { margin: auto; position: absolute; top: 0; left: 0; right: 0; bottom: 0; } .relative.btn-hidden #btnImg { display: none; }
$(function() { $("#btnImg").css({ opacity: ".65" }); $("#btnImg").hover( function() { $(this) .stop() .animate({ opacity: "1" }); }, function() { $(this) .stop() .animate({ opacity: ".65" }); } ); $("#btnImg").css("cursor", "pointer"); var $outer = $('.relative'); $("#btnImg").on('click', function() { $("#changeImg").attr({ src: "02.gif" }); $outer.addClass('btn-hidden'); $(this).css("cursor", "auto"); setTimeout(function() { $("#changeImg").attr({ src: "01.jpg" }); $("#btnImg").attr({ src: "03.png" }); $("#btnImg").css({ opacity: ".65" }); $outer.removeClass('btn-hidden'); $("#btnImg").css("cursor", "pointer"); }, 5000); }); });

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

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

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

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

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

m.ts10806

2018/11/12 11:58

作業依頼に等しいですし単に「見て直して」だけだと回答者の負担が大きすぎます。同じプロジェクトの新人が書いたコードならまだしも、赤の他人の全く馴染みのない要件のコードです。想定通り動いているのならそれでよしとし、もし質問をしたいのであればもっとポイントを絞るべきではないでしょうか
guest

回答1

0

hover
非同期
click
setTimeout
内の処理を全て外で関数にして、
var
onloadのcss変更
は上にまとめるのはどうでしょうか?

投稿2018/11/12 14:43

akihiro3

総合スコア955

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問