[PR #1735] chore: improve grid focus and hover event handling #13336

New issue

Closed

opened 2026-03-24 21:46:08 +00:00 by mirror · 0 comments

mirror commented

2026-03-24 21:46:08 +00:00

Owner

Original Pull Request: https://github.com/AppFlowy-IO/AppFlowy/pull/1735

State: closed
Merged: Yes

As a result of a previous change to fix cell accessories on URL cells, the 'open as a page' accessory for primary field cells no longer appear when hovering over the row, add it back!
ChangeNotifierProvider doesn't recreate the notifier when CellContainer is rebuilt, which causes the bug. Therefore, make the notifier also rebuild with CellContainer.
remove unnecessary extra Stack and Inkwell widgets used to detect mouse presses in select option and checklist cells by porting to GridCellState.

resolves #1543

**Original Pull Request:** https://github.com/AppFlowy-IO/AppFlowy/pull/1735 **State:** closed **Merged:** Yes --- - As a result of a previous change to fix cell accessories on URL cells, the 'open as a page' accessory for primary field cells no longer appear when hovering over the row, add it back! - `ChangeNotifierProvider` doesn't recreate the notifier when `CellContainer` is rebuilt, which causes the bug. Therefore, make the notifier also rebuild with `CellContainer`. - remove unnecessary extra `Stack` and `Inkwell` widgets used to detect mouse presses in select option and checklist cells by porting to `GridCellState`. resolves #1543

Rows
Columns