MDCMigration: Style data table row heights for MDC migration.

tensorflow · Oct 3, 2023 · 8324f32 · 8324f32
1 parent 9757188
commit 8324f32
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/tensorboard/webapp/metrics/views/card_renderer/scalar_card_data_table.scss b/tensorboard/webapp/metrics/views/card_renderer/scalar_card_data_table.scss
@@ -14,6 +14,11 @@ limitations under the License.
 ==============================================================================*/
 $_circle-size: 12px;
 
+tb-data-table-content-cell,
+tb-data-table-header-cell {
+  padding: 4px;
+}
+
 .row-circle {
   height: $_circle-size;
   width: $_circle-size;

diff --git a/tensorboard/webapp/runs/views/runs_table/runs_data_table.scss b/tensorboard/webapp/runs/views/runs_table/runs_data_table.scss
@@ -48,11 +48,13 @@ $_arrow_size: 16px;
 
 tb-data-table-content-row,
 tb-data-table-header-cell {
-  height: 44px;
+  // Match the size of the touch targets for the checkbox.
+  height: 48px;
 }
 
 tb-data-table-content-cell,
 tb-data-table-header-cell {
+  padding: 0 4px;
   vertical-align: middle;
   @include tb-theme-foreground-prop(border-bottom, border, 1px solid);
 

diff --git a/tensorboard/webapp/widgets/data_table/content_cell_component.scss b/tensorboard/webapp/widgets/data_table/content_cell_component.scss
@@ -15,8 +15,8 @@ limitations under the License.
 
 :host {
   display: table-cell;
-  padding: 4px;
 }
+
 .cell {
   align-items: center;
   display: flex;

diff --git a/tensorboard/webapp/widgets/data_table/header_cell_component.scss b/tensorboard/webapp/widgets/data_table/header_cell_component.scss
@@ -21,7 +21,6 @@ $_icon_size: 12px;
 
 :host {
   display: table-cell;
-  padding: 4px;
   vertical-align: bottom;
 
   &:hover {