Fix datagrid lines (opensearch-project#1250)

Signed-off-by: Miki <miki@amazon.com>

CHANGELOG.md

@@ -39,6 +39,7 @@
 - Prevent collapsable button of a resizable from adding to the height when hidden ([#1241](https://github.com/opensearch-project/oui/pull/1241))
 - Prevent action buttons of OuiDataGridCell from moving the content ([#1224](https://github.com/opensearch-project/oui/pull/1224))
 - Fix vertical alignment and font weight of breadcrumbs that are buttons or links ([#1232](https://github.com/opensearch-project/oui/pull/1232))
+- Fix datagrid lines ([#1250](https://github.com/opensearch-project/oui/pull/1250))
 
 ### 🚞 Infrastructure
 

src/components/datagrid/_data_grid_data_row.scss

@@ -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;
@@ -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;
   }