回答編集履歴
11
テキスト修正
test
CHANGED
@@ -132,7 +132,7 @@
|
|
132
132
|
|
133
133
|
|
134
134
|
|
135
|
-
に対応したのが、以下の
|
135
|
+
に対応したのが、以下の部分
|
136
136
|
|
137
137
|
|
138
138
|
|
10
テキスト修正
test
CHANGED
@@ -156,7 +156,7 @@
|
|
156
156
|
|
157
157
|
|
158
158
|
|
159
|
-
について修正した
|
159
|
+
について修正した箇所は、行を追加するかどうか判定する if 条件
|
160
160
|
|
161
161
|
|
162
162
|
|
@@ -168,7 +168,7 @@
|
|
168
168
|
|
169
169
|
|
170
170
|
|
171
|
-
上記の修正案1では、ご質問に挙げられている2点のほか、以下の修正もしています。
|
171
|
+
の部分です。上記の修正案1では、ご質問に挙げられている2点のほか、以下の修正もしています。
|
172
172
|
|
173
173
|
|
174
174
|
|
@@ -178,7 +178,7 @@
|
|
178
178
|
|
179
179
|
|
180
180
|
|
181
|
-
さらに、もう少し見通しよく
|
181
|
+
さらに、もう少し見通しよくするならば、 selectでchange が発生したときに、行を追加する条件は
|
182
182
|
|
183
183
|
|
184
184
|
|
@@ -208,7 +208,7 @@
|
|
208
208
|
|
209
209
|
|
210
210
|
|
211
|
-
以下の修正案2は上記の `canAddRow` を使って、行を追加
|
211
|
+
以下の修正案2は上記の `canAddRow` を使って、行を追加できる`select` がどうかを判定する部分を整理したものです。
|
212
212
|
|
213
213
|
|
214
214
|
|
9
テキスト修正
test
CHANGED
@@ -138,9 +138,9 @@
|
|
138
138
|
|
139
139
|
```javascript
|
140
140
|
|
141
|
-
|
141
|
+
if ($('.changeList').index(this) !== $('.changeList').length-1)
|
142
|
-
|
142
|
+
|
143
|
-
|
143
|
+
return;
|
144
144
|
|
145
145
|
```
|
146
146
|
|
@@ -178,7 +178,7 @@
|
|
178
178
|
|
179
179
|
|
180
180
|
|
181
|
-
さらに、少し見通しよくしようとすれば、select でchange が発生したときに、行を追加する条件は
|
181
|
+
さらに、もう少し見通しよくしようとすれば、select でchange が発生したときに、行を追加する条件は
|
182
182
|
|
183
183
|
|
184
184
|
|
8
テキスト修正
test
CHANGED
@@ -182,11 +182,11 @@
|
|
182
182
|
|
183
183
|
|
184
184
|
|
185
|
-
そのchange の発生したselectが表の最下行のものであって、かつ、選択された値が `select-default` ではないこと
|
185
|
+
- そのchange の発生したselectが表の最下行のものであって、かつ、選択された値が `select-default` ではないこと
|
186
|
-
|
187
|
-
|
188
|
-
|
186
|
+
|
187
|
+
|
188
|
+
|
189
|
-
なので、この条件を満たすかどうかの判定をひとまとめに関数にしてしま
|
189
|
+
なので、この条件を満たすかどうかの判定をひとまとめに関数にして、条件を手短かに書ける if が一つあれば済むようにします。以下では、この判定のための関数をJQuery のプラグインとして書いた例です。
|
190
190
|
|
191
191
|
|
192
192
|
|
7
テキスト修正
test
CHANGED
@@ -120,7 +120,7 @@
|
|
120
120
|
|
121
121
|
|
122
122
|
|
123
|
-
上記の修正案1のコードの中で
|
123
|
+
上記の修正案1のコードの中で、ご質問にある1点目の
|
124
124
|
|
125
125
|
|
126
126
|
|
@@ -132,7 +132,7 @@
|
|
132
132
|
|
133
133
|
|
134
134
|
|
135
|
-
に対応したのが、以下の行
|
135
|
+
に対応したのが、以下の行
|
136
136
|
|
137
137
|
|
138
138
|
|
@@ -144,6 +144,8 @@
|
|
144
144
|
|
145
145
|
```
|
146
146
|
|
147
|
+
です。2点目の
|
148
|
+
|
147
149
|
|
148
150
|
|
149
151
|
> また、「選択してください」を選択したときは、新規セレクトボックスを追加しないようにするため「選択してください」(name="select-default")以外を選択時のみに、
|
6
テキスト修正
test
CHANGED
@@ -166,7 +166,7 @@
|
|
166
166
|
|
167
167
|
|
168
168
|
|
169
|
-
上記の修正案1では、
|
169
|
+
上記の修正案1では、ご質問に挙げられている2点のほか、以下の修正もしています。
|
170
170
|
|
171
171
|
|
172
172
|
|
5
テキスト修正
test
CHANGED
@@ -108,7 +108,7 @@
|
|
108
108
|
|
109
109
|
|
110
110
|
|
111
|
-
動作確認のため、以下の jsFiddle に
|
111
|
+
動作確認のため、上記のコードは、以下の jsFiddle の修正案1のほうに上げています。
|
112
112
|
|
113
113
|
|
114
114
|
|
4
テキスト修正
test
CHANGED
@@ -32,7 +32,7 @@
|
|
32
32
|
|
33
33
|
|
34
34
|
|
35
|
-
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option` を取ってくるための
|
35
|
+
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option` を取ってくるためのセレクタが正しくないです。
|
36
36
|
|
37
37
|
|
38
38
|
|
3
テキスト修正
test
CHANGED
@@ -184,7 +184,7 @@
|
|
184
184
|
|
185
185
|
|
186
186
|
|
187
|
-
なので、この条件をひとまとめに関数にしてしまいます。以下では、この関数をJQuery のプラグインとして書いた例です。
|
187
|
+
なので、この条件を満たすかどうかの判定をひとまとめに関数にしてしまいます。以下では、この関数をJQuery のプラグインとして書いた例です。
|
188
188
|
|
189
189
|
|
190
190
|
|
2
テキスト修正
test
CHANGED
@@ -32,7 +32,7 @@
|
|
32
32
|
|
33
33
|
|
34
34
|
|
35
|
-
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option` の
|
35
|
+
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option` を取ってくるための `$(・・・)` が正しくないです。
|
36
36
|
|
37
37
|
|
38
38
|
|
1
テキスト修正
test
CHANGED
@@ -32,7 +32,7 @@
|
|
32
32
|
|
33
33
|
|
34
34
|
|
35
|
-
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option のセレクタが正しくないです。
|
35
|
+
また、 `select-default` と比較すべき、 `.val()` で値を取得する `option` のセレクタが正しくないです。
|
36
36
|
|
37
37
|
|
38
38
|
|
@@ -108,15 +108,11 @@
|
|
108
108
|
|
109
109
|
|
110
110
|
|
111
|
-
動作確認のため、jsFiddle にも上げています。
|
111
|
+
動作確認のため、以下の jsFiddle にも上げています。
|
112
|
-
|
113
|
-
|
114
|
-
|
112
|
+
|
113
|
+
|
114
|
+
|
115
|
-
- **修正前:** [https://jsfiddle.net/jun68ykt/0w74vueh/1/](https://jsfiddle.net/jun68ykt/0w74vueh/1/)
|
115
|
+
- **修正前:** [https://jsfiddle.net/jun68ykt/0w74vueh/1/](https://jsfiddle.net/jun68ykt/0w74vueh/1/)
|
116
|
-
|
117
|
-
|
118
|
-
|
119
|
-
|
120
116
|
|
121
117
|
|
122
118
|
|