Skip to content

Commit 7594877

Browse files
fix(table): pr edits
- adds table mods to button passthroughs for fallbacks - removes transition on header cells - removes unnecessary commas in some property definitions
1 parent 6f1f7b2 commit 7594877

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

components/table/index.css

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -133,10 +133,10 @@
133133
--mod-button-background-color-hover: transparent;
134134
--mod-button-background-color-focus: transparent;
135135
--mod-button-background-color-down: transparent;
136-
--mod-button-content-color-default: var(--spectrum-table-header-text-color);
137-
--mod-button-content-color-hover: var(--spectrum-table-header-text-color);
138-
--mod-button-content-color-focus: var(--spectrum-table-header-text-color);
139-
--mod-button-content-color-down: var(--spectrum-table-header-text-color);
136+
--mod-button-content-color-default: var(--mod-table-header-text-color, var(--spectrum-table-header-text-color));
137+
--mod-button-content-color-hover: var(--mod-table-header-text-color, var(--spectrum-table-header-text-color));
138+
--mod-button-content-color-focus: var(--mod-table-header-text-color, var(--spectrum-table-header-text-color));
139+
--mod-button-content-color-down: var(--mod-table-header-text-color, var(--spectrum-table-header-text-color));
140140
--mod-button-edge-to-text: var(--spectrum-table-cell-inline-spacing);
141141
--mod-button-padding-label-to-icon: var(--spectrum-table-visual-to-text);
142142
--mod-button-border-radius: calc(var(--mod-table-border-radius, var(--spectrum-table-focused-cell-border-radius)) - var(--mod-table-border-width, var(--spectrum-table-border-width)));
@@ -197,8 +197,7 @@
197197
--spectrum-table-header-background-color: var(--mod-table-header-background-color-scrollable, var(--spectrum-gray-75));
198198
}
199199

200-
.spectrum-Table-row,
201-
.spectrum-Table-headRow {
200+
.spectrum-Table-row {
202201
&.is-focused,
203202
&:focus-visible,
204203
&:focus {
@@ -349,7 +348,6 @@
349348
background-color: var(--mod-table-header-background-color, var(--spectrum-table-header-background-color));
350349
border-block-start: var(--mod-table-border-width, var(--spectrum-table-border-width)) solid var(--highcontrast-table-border-color, var(--mod-table-border-color, var(--spectrum-table-border-color)));
351350

352-
transition: color var(--highcontrast-table-transition-duration, var(--mod-table-transition-duration, var(--spectrum-table-transition-duration))) ease-in-out;
353351
cursor: var(--mod-table-cursor-header-default, initial);
354352
outline: 0;
355353

@@ -914,10 +912,10 @@
914912
border: 0;
915913
color: var(--spectrum-table-icon-color);
916914

917-
transform: var(--spectrum-logical-rotation,);
915+
transform: var(--spectrum-logical-rotation);
918916

919917
&[aria-expanded="true"] {
920-
transform: var(--spectrum-logical-rotation,) rotate(90deg);
918+
transform: var(--spectrum-logical-rotation) rotate(90deg);
921919
}
922920

923921
&::after {

0 commit comments

Comments
 (0)