Skip to content

Commit

Permalink
Update Index table colors (Shopify#10755)
Browse files Browse the repository at this point in the history
### WHY are these changes introduced?

Fixes Shopify#10362

### WHAT is this pull request doing?

This updates some Index table row colors as well as the color value for
the `color-bg-surface-selected` token
  • Loading branch information
aveline committed Sep 29, 2023
1 parent d0a2898 commit cfaefac
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions polaris-react/src/components/IndexTable/IndexTable.scss
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ $loading-panel-height: 53px;
.TableHeading-second,
.TableCell-first,
.TableCell-first + .TableCell {
background-color: var(--p-color-bg-primary-subdued-selected);
background-color: var(--p-color-bg-surface-selected);
}

/* stylelint-disable-next-line selector-max-class -- generated by polaris-migrator DO NOT COPY */
Expand Down Expand Up @@ -359,7 +359,7 @@ $loading-panel-height: 53px;
.TableCell-first,
.TableCell-first + .TableCell,
.TableCell:last-child {
background-color: var(--p-color-bg-primary-subdued-hover);
background-color: var(--p-color-bg-surface-hover);
}

/* stylelint-disable-next-line selector-max-class, selector-max-specificity -- generated by polaris-migrator DO NOT COPY */
Expand Down
2 changes: 1 addition & 1 deletion polaris-tokens/src/themes/base/color.ts
Original file line number Diff line number Diff line change
Expand Up @@ -433,7 +433,7 @@ export const color: {
value: colorsExperimental.gray[4],
},
'color-bg-surface-selected': {
value: colorsExperimental.gray[3],
value: colorsExperimental.gray[6],
},
'color-bg-fill-success': {
value: colorsExperimental.green[12],
Expand Down

0 comments on commit cfaefac

Please sign in to comment.