Skip to content

Commit

Permalink
Fix datagrid lines (#1250) (#1253)
Browse files Browse the repository at this point in the history
(cherry picked from commit 543435a)

Signed-off-by: Miki <miki@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
  • Loading branch information
1 parent 466145b commit b01a3a9
Showing 1 changed file with 17 additions and 15 deletions.
32 changes: 17 additions & 15 deletions src/components/datagrid/_data_grid_data_row.scss
Original file line number Diff line number Diff line change
Expand Up @@ -166,21 +166,6 @@
}

.ouiDataGridRowCell {
box-shadow: none !important; // sass-lint:disable-line no-important

/* ToDo: Remove unnecessary logic: the old beta theme has
* evolved and the statement below about the `Next` theme
* is not valid.
*/
// Remove default .ouiButtonIcon--fill border
// This way we don't need to animate the border that is inexistent in Next and unnecessary for a fill button
border: none;

&-isActive {
margin-left: $ouiDataGridCellPaddingM;
width: $ouiSizeM;
}

&__actionButtonIcon,
&__expandButtonIcon {
height: $ouiSizeM;
Expand All @@ -190,6 +175,23 @@
transition: none; // Have to take out the generic transition so it is instantaneous on focus
}

&__expandButtonIcon {
box-shadow: none !important; // sass-lint:disable-line no-important

/* ToDo: Remove unnecessary logic: the old beta theme has
* evolved and the statement below about the `Next` theme
* is not valid.
*/
// Remove default .ouiButtonIcon--fill border
// This way we don't need to animate the border that is inexistent in Next and unnecessary for a fill button
border: none;

&-isActive {
margin-left: $ouiDataGridCellPaddingM;
width: $ouiSizeM;
}
}

&__expandButton > * {
z-index: 2;
}
Expand Down

0 comments on commit b01a3a9

Please sign in to comment.