回答編集履歴
3
修正
test
CHANGED
|
@@ -6,7 +6,9 @@
|
|
|
6
6
|
|
|
7
7
|
セレクタの指定が違えばそれも含めて「スペルミス」と言えます。
|
|
8
8
|
|
|
9
|
-
- `img_none`という変数はどこにも定義されていません。
|
|
9
|
+
- `img_none`という変数はどこにも定義されていません。定義されていない変数を利用するのもある意味「スペルミス」と言えます。
|
|
10
|
+
|
|
11
|
+
定義したものをコピペすれば避けられる話です。
|
|
10
12
|
|
|
11
13
|
- ` ↓ここからの記述がわからないです・・・` ←実際にこのままコード内に書いてあるなら動かないのは当然ですが、せめてコメントアウトしてください。
|
|
12
14
|
|
2
修正
test
CHANGED
|
@@ -15,3 +15,5 @@
|
|
|
15
15
|
- jQueryでクラスを指定するなら配列回してaddEventするのではなくjQueryで`$('セレクタ').on('イベント',function(){/*処理*/})`で良いのでは？
|
|
16
16
|
|
|
17
17
|
混ざっていると実装のブレが出てよろしくありません。
|
|
18
|
+
|
|
19
|
+
- つまり「クラスを付与する」も[jQueryの記法](https://api.jquery.com/addClass/)でやるべきです
|
1
修正
test
CHANGED
|
@@ -11,3 +11,7 @@
|
|
|
11
11
|
- ` ↓ここからの記述がわからないです・・・` ←実際にこのままコード内に書いてあるなら動かないのは当然ですが、せめてコメントアウトしてください。
|
|
12
12
|
|
|
13
13
|
- ご自身はこのコードをどこまで把握しているのでしょうか。
|
|
14
|
+
|
|
15
|
+
- jQueryでクラスを指定するなら配列回してaddEventするのではなくjQueryで`$('セレクタ').on('イベント',function(){/*処理*/})`で良いのでは？
|
|
16
|
+
|
|
17
|
+
混ざっていると実装のブレが出てよろしくありません。
|