Skip to content

Commit

Permalink
fix(DataTable): fix small toolbar sizing and color bug (#10296)
Browse files Browse the repository at this point in the history
* feat: add v11 table stories

* fix: small toolbar/batch styles + stories

* fix: overflow menu icon color bug

* fix: table next tests

* fix: v11 table toolbar styles

* fix: table stories

* fix: v11 search icon spacing
  • Loading branch information
jnm2377 authored Jan 31, 2022
1 parent 430693f commit 60d7518
Show file tree
Hide file tree
Showing 23 changed files with 3,175 additions and 467 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -453,6 +453,7 @@
.#{$prefix}--table-toolbar--small,
.#{$prefix}--table-toolbar--sm {
height: rem(32px);
min-height: rem(32px);

.#{$prefix}--toolbar-search-container-expandable,
.#{$prefix}--toolbar-search-container-persistent {
Expand Down Expand Up @@ -536,6 +537,17 @@

background-color: transparent;
}

.#{$prefix}--overflow-menu.#{$prefix}--toolbar-action {
width: rem(32px);
//:after element is 2rem but w/o this trigger button is for some reason 28px
min-width: rem(32px);
height: rem(32px);
}

.#{$prefix}--toolbar-content {
height: rem(32px);
}
}

.#{$prefix}--search--disabled .#{$prefix}--search-magnifier-icon:hover {
Expand Down Expand Up @@ -590,6 +602,12 @@
overflow: hidden;
height: rem(32px);
}

// V11: remove --small selector block
.#{$prefix}--table-toolbar--small .#{$prefix}--batch-summary,
.#{$prefix}--table-toolbar--sm .#{$prefix}--batch-summary {
min-height: 2rem;
}
}

@include exports('data-table-v2-action') {
Expand Down
Loading

0 comments on commit 60d7518

Please sign in to comment.