From f81d5eb23ea0dabb5fdd53923174bcbda4d49fe2 Mon Sep 17 00:00:00 2001 From: Richard Shiue <71320345+richardshiue@users.noreply.github.com> Date: Thu, 8 Dec 2022 14:18:57 +0800 Subject: [PATCH] fix: location of popover wrt the cell (#1545) --- .../widgets/cell/checklist_cell/checklist_cell.dart | 10 +++++----- .../cell/select_option_cell/select_option_cell.dart | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/checklist_cell/checklist_cell.dart b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/checklist_cell/checklist_cell.dart index 23f6cc43d3..47d6342f46 100644 --- a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/checklist_cell/checklist_cell.dart +++ b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/checklist_cell/checklist_cell.dart @@ -40,10 +40,7 @@ class GridChecklistCellState extends State { alignment: AlignmentDirectional.center, fit: StackFit.expand, children: [ - Padding( - padding: GridSize.cellContentInsets, - child: _wrapPopover(const ChecklistProgressBar()), - ), + _wrapPopover(const ChecklistProgressBar()), InkWell(onTap: () => _popover.show()), ], ), @@ -66,7 +63,10 @@ class GridChecklistCellState extends State { ); }, onClose: () => widget.onCellEditing.value = false, - child: child, + child: Padding( + padding: GridSize.cellContentInsets, + child: child, + ), ); } } diff --git a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/select_option_cell/select_option_cell.dart b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/select_option_cell/select_option_cell.dart index bf4966dd51..e228f83c16 100644 --- a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/select_option_cell/select_option_cell.dart +++ b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/select_option_cell/select_option_cell.dart @@ -168,10 +168,7 @@ class _SelectOptionWrapState extends State { alignment: AlignmentDirectional.center, fit: StackFit.expand, children: [ - Padding( - padding: GridSize.cellContentInsets, - child: _wrapPopover(child), - ), + _wrapPopover(child), InkWell(onTap: () => _popover.show()), ], ); @@ -196,7 +193,10 @@ class _SelectOptionWrapState extends State { ); }, onClose: () => widget.onFocus?.call(false), - child: child, + child: Padding( + padding: GridSize.cellContentInsets, + child: child, + ), ); }