forked from SymfonyCasts/symfony-ux
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathvalues-use-value-exclusively.diff
54 lines (51 loc) · 1.59 KB
/
values-use-value-exclusively.diff
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
diff --git a/assets/controllers/color-square_controller.js b/assets/controllers/color-square_controller.js
index 3f8d698..19b3e0a 100644
--- a/assets/controllers/color-square_controller.js
+++ b/assets/controllers/color-square_controller.js
@@ -1,8 +1,6 @@
import { Controller } from 'stimulus';
export default class extends Controller {
- selectedColorId = null;
-
static targets = ['colorSquare', 'select']
static values = {
colorId: Number
@@ -13,7 +11,7 @@ export default class extends Controller {
}
selectColor(event) {
- this.setSelectedColor(event.currentTarget.dataset.colorId)
+ this.colorIdValue = event.currentTarget.dataset.colorId;
}
colorIdValueChanged() {
@@ -27,31 +25,4 @@ export default class extends Controller {
}
});
}
-
- setSelectedColor(clickedColorId) {
- if (clickedColorId === this.selectedColorId) {
- this.findSelectedColorSquare().classList.remove('selected');
-
- this.selectedColorId = null;
- this.selectTarget.value = '';
-
- return;
- }
-
- this.selectedColorId = clickedColorId;
-
- this.colorSquareTargets.forEach((element) => {
- element.classList.remove('selected');
- });
-
- this.findSelectedColorSquare().classList.add('selected');
- this.selectTarget.value = this.selectedColorId;
- }
-
- /**
- * @return {Element|null}
- */
- findSelectedColorSquare() {
- return this.colorSquareTargets.find((element) => element.dataset.colorId == this.selectedColorId);
- }
}