Skip to content

Commit 41371cf

Browse files
committed
feat: Add data-action-* hover classes
Instead of relying on fa- icon classes for the action color, we use the data-action attribute
1 parent 6921660 commit 41371cf

1 file changed

Lines changed: 22 additions & 6 deletions

File tree

  • backend/app/assets/stylesheets/spree/backend/shared

backend/app/assets/stylesheets/spree/backend/shared/_tables.scss

Lines changed: 22 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,28 +92,44 @@ table {
9292
}
9393
}
9494

95-
.fa-trash:hover, .fa-void:hover, .fa-failure:hover {
95+
.fa-trash:hover,
96+
.fa-void:hover,
97+
.fa-failure:hover,
98+
[data-action="remove"]:hover {
9699
color: theme-color("danger");
97100
}
98101

99-
.fa-cancel:hover {
102+
.fa-cancel:hover,
103+
[data-action="cancel"]:hover {
100104
color: theme-color("warning");
101105
}
102106

103-
.fa-edit:hover, .fa-capture:hover, .fa-ok:hover, .fa-plus:hover,
104-
.fa-save:hover, .fa-arrows-h:hover, .fa-check:hover {
107+
.fa-edit:hover,
108+
.fa-capture:hover,
109+
.fa-ok:hover,
110+
.fa-plus:hover,
111+
.fa-save:hover,
112+
.fa-arrows-h:hover,
113+
.fa-check:hover,
114+
[data-action="save"]:hover,
115+
[data-action="edit"]:hover,
116+
[data-action="capture"]:hover,
117+
[data-action="add"]:hover,
118+
[data-action="new"]:hover {
105119
color: theme-color("success");
106120
}
107121

108-
.fa-copy:hover {
122+
.fa-copy:hover,
123+
[data-action="clone"]:hover {
109124
color: theme-color("warning");
110125
}
111126

112127
.fa-thumbs-up:hover {
113128
color: theme-color("success");
114129
}
115130

116-
.fa-thumbs-down:hover {
131+
.fa-thumbs-down:hover,
132+
[data-action="void"]:hover {
117133
color: theme-color("danger");
118134
}
119135
}

0 commit comments

Comments
 (0)