From 816f28e3d38ae1274e7ebadd17e2f3893874e349 Mon Sep 17 00:00:00 2001 From: qianlifeng Date: Sun, 13 Jul 2025 22:28:41 +0800 Subject: [PATCH] refactor(ui): adjust DataTable row heights for improved layout * Reduced `headingRowHeight` to 36 for consistency across tables. * Set `dataRowMinHeight` and `dataRowMaxHeight` to 36 to enhance visual uniformity. * Updated tooltip labels to use `tr("ui_operation")` for better localization. --- .../plugin/wox_setting_plugin_table_view.dart | 14 +++++++++----- .../setting/views/wox_setting_data_view.dart | 4 +++- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/wox.ui.flutter/wox/lib/components/plugin/wox_setting_plugin_table_view.dart b/wox.ui.flutter/wox/lib/components/plugin/wox_setting_plugin_table_view.dart index f55df023..fa73eb61 100644 --- a/wox.ui.flutter/wox/lib/components/plugin/wox_setting_plugin_table_view.dart +++ b/wox.ui.flutter/wox/lib/components/plugin/wox_setting_plugin_table_view.dart @@ -444,7 +444,9 @@ class WoxSettingPluginTable extends WoxSettingPluginItem { columnSpacing: columnSpacing, horizontalMargin: 5, clipBehavior: Clip.hardEdge, - headingRowHeight: 48, + headingRowHeight: 36, + dataRowMinHeight: 36, + dataRowMaxHeight: 36, headingRowColor: material.WidgetStateProperty.resolveWith((states) => fromCssColor(WoxThemeUtil.instance.currentTheme.value.actionItemActiveBackgroundColor)), border: TableBorder.all(color: fromCssColor(WoxThemeUtil.instance.currentTheme.value.previewSplitLineColor)), columns: [ @@ -472,7 +474,7 @@ class WoxSettingPluginTable extends WoxSettingPluginItem { label: columnWidth( column: PluginSettingValueTableColumn.fromJson({ "Key": "Operation", - "Label": tr("operation"), + "Label": tr("ui_operation"), "Tooltip": "", "Width": operationWidth.toInt(), "Type": PluginSettingValueType.pluginSettingValueTableColumnTypeText, @@ -481,9 +483,9 @@ class WoxSettingPluginTable extends WoxSettingPluginItem { isHeader: false, isOperation: true, child: material.Tooltip( - message: tr("operation"), + message: tr("ui_operation"), child: Text( - tr("operation"), + tr("ui_operation"), overflow: TextOverflow.ellipsis, maxLines: 1, style: TextStyle( @@ -565,7 +567,9 @@ class WoxSettingPluginTable extends WoxSettingPluginItem { child: material.DataTable( columnSpacing: columnSpacing, horizontalMargin: 5, - headingRowHeight: 40, + headingRowHeight: 36, + dataRowMinHeight: 36, + dataRowMaxHeight: 36, headingRowColor: material.WidgetStateProperty.resolveWith((states) => fromCssColor(WoxThemeUtil.instance.currentTheme.value.actionItemActiveBackgroundColor)), border: TableBorder.all(color: fromCssColor(WoxThemeUtil.instance.currentTheme.value.previewSplitLineColor)), columns: [ diff --git a/wox.ui.flutter/wox/lib/modules/setting/views/wox_setting_data_view.dart b/wox.ui.flutter/wox/lib/modules/setting/views/wox_setting_data_view.dart index 731cc197..0cf099ef 100644 --- a/wox.ui.flutter/wox/lib/modules/setting/views/wox_setting_data_view.dart +++ b/wox.ui.flutter/wox/lib/modules/setting/views/wox_setting_data_view.dart @@ -145,7 +145,9 @@ class WoxSettingDataView extends WoxSettingBaseView { return material.DataTable( columnSpacing: 10, horizontalMargin: 5, - headingRowHeight: 48, + headingRowHeight: 36, + dataRowMinHeight: 36, + dataRowMaxHeight: 36, headingRowColor: material.WidgetStateProperty.resolveWith((states) => fromCssColor(WoxThemeUtil.instance.currentTheme.value.actionItemActiveBackgroundColor)), border: TableBorder.all(color: fromCssColor(WoxThemeUtil.instance.currentTheme.value.previewSplitLineColor)), columns: [