From 873a46e4430c9db9c1770933dffac4c7713ea3de Mon Sep 17 00:00:00 2001
From: Eakam <67077705+Eakam1007@users.noreply.github.com>
Date: Wed, 21 Dec 2022 06:11:49 -0500
Subject: [PATCH] fix: no focus border on date cells (#1469)

* fix: no focus border on date cells

* fix: remove redundant import

* refactor: use existing functionality from GridCellWidget for focus border

* refactor: use requestBeginFocus override instead of GestureDetector
---
 .../widgets/cell/date_cell/date_cell.dart     | 23 +++++++++----------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/date_cell/date_cell.dart b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/date_cell/date_cell.dart
index 43fdcd647c..64813a5fd5 100644
--- a/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/date_cell/date_cell.dart
+++ b/frontend/app_flowy/lib/plugins/grid/presentation/widgets/cell/date_cell/date_cell.dart
@@ -70,17 +70,13 @@ class _DateCellState extends GridCellState<GridDateCell> {
             constraints: BoxConstraints.loose(const Size(260, 500)),
             margin: EdgeInsets.zero,
             child: SizedBox.expand(
-              child: GestureDetector(
-                behavior: HitTestBehavior.opaque,
-                onTap: () => _popover.show(),
-                child: Align(
-                  alignment: alignment,
-                  child: Padding(
-                    padding: GridSize.cellContentInsets,
-                    child: FlowyText.medium(
-                      state.dateStr,
-                      overflow: TextOverflow.ellipsis,
-                    ),
+              child: Align(
+                alignment: alignment,
+                child: Padding(
+                  padding: GridSize.cellContentInsets,
+                  child: FlowyText.medium(
+                    state.dateStr,
+                    overflow: TextOverflow.ellipsis,
                   ),
                 ),
               ),
@@ -108,7 +104,10 @@ class _DateCellState extends GridCellState<GridDateCell> {
   }
 
   @override
-  void requestBeginFocus() {}
+  void requestBeginFocus() {
+    _popover.show();
+    widget.onCellEditing.value = true;
+  }
 
   @override
   String? onCopy() => _cellBloc.state.dateStr;