teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

4

かな

2017/05/01 06:32

投稿

退会済みユーザー
answer CHANGED
@@ -30,4 +30,4 @@
30
30
  追記
31
31
 
32
32
  すみません。jQueryで動作検証したため普通のhtmlのイベントだとthisがspanに紐づかないということに気づきませんでした。
33
- eventオブジェクトを使う方法で直しても良いのですが、楽な方法でなおしました。動作確認できているのでご参考にしてもらえレバと思います。
33
+ eventオブジェクトを使う方法で直しても良いのですが、楽な方法でなおしました。動作確認できているのでご参考にしてもらえればと思います。

3

コード修正

2017/05/01 06:32

投稿

退会済みユーザー
answer CHANGED
@@ -5,15 +5,29 @@
5
5
  コードを読ませていただいたところ2種類の関数とデザインだけ用意すればいいようなので
6
6
 
7
7
  ```javascript
8
- function check() {
8
+ function check(obj) {
9
- this.classList.remove("kuro");
9
+ obj.classList.remove("hide");
10
- this.onclick = uncheck;
10
+ obj.onclick = new Function("uncheck(this)");
11
11
  }
12
- function uncheck() {
12
+ function uncheck(obj) {
13
- this.classList.add("kuro");
13
+ obj.classList.add("hide");
14
- this.onclick = check;
14
+ obj.onclick = new Function("check(this)");
15
15
  }
16
16
  ```
17
- としてhtmlのid="kuro01"をclass="kuro"、onclick=check、cssのセレクタを#kuro~から.kuroを一つに変えるだけで動くと思います。#siro~は全く必要ありません。
18
17
 
18
+ ```html
19
+ <span onclick="check(this)" class="kuro hide">〜〜〜〜</span>
20
+ ```
21
+
22
+ ```css
23
+ .kuro {cursor: pointer;}
24
+ .kuro.hide{background-color: black;}
25
+ ```
26
+
19
- もし今後もWEBページを作る仕事をする必要があり、ちゃんとWEBページの作り方を勉強したことないのであれば一度ちゃんと勉強されてから作ったほうが書いてるときに大変にならずよいと思います。
27
+ もし今後もWEBページを作る仕事をする必要があり、ちゃんとWEBページの作り方を勉強したことないのであれば一度ちゃんと勉強されてから作ったほうが書いてるときに大変にならずよいと思います。
28
+
29
+
30
+ 追記
31
+
32
+ すみません。jQueryで動作検証したため普通のhtmlのイベントだとthisがspanに紐づかないということに気づきませんでした。
33
+ eventオブジェクトを使う方法で直しても良いのですが、楽な方法でなおしました。動作確認できているのでご参考にしてもらえレバと思います。

2

typo

2017/05/01 06:32

投稿

退会済みユーザー
answer CHANGED
@@ -7,7 +7,7 @@
7
7
  ```javascript
8
8
  function check() {
9
9
  this.classList.remove("kuro");
10
- element.onclick = uncheck;
10
+ this.onclick = uncheck;
11
11
  }
12
12
  function uncheck() {
13
13
  this.classList.add("kuro");

1

情報欠落修正

2017/04/16 08:17

投稿

退会済みユーザー
answer CHANGED
@@ -14,6 +14,6 @@
14
14
  this.onclick = check;
15
15
  }
16
16
  ```
17
- htmlのid="kuro01"をclass="kuro"、cssのセレクタを#kuro~から.kuroを一つに変えるだけで動くと思います。#siro~は全く必要ありません。
17
+ としてhtmlのid="kuro01"をclass="kuro"、onclick=check、cssのセレクタを#kuro~から.kuroを一つに変えるだけで動くと思います。#siro~は全く必要ありません。
18
18
 
19
19
  もし今後もWEBページを作る仕事をする必要があり、ちゃんとWEBページの作り方を勉強したことないのであれば一度ちゃんと勉強されてから作ったほうが書いてるときに大変にならずよいと思います。