fix(document): één klik wisselt weer tussen tabel en tekst (#2461) #2482

Merged

brenno merged 1 commit from fix/table-een-klik into main

2026-10-19 17:27:11 +00:00

brenno commented

2026-10-19 17:26:16 +00:00

Owner

Samenvatting

#2461: één klik tussen tabelcel en aangrenzende tekst werkte niet — de
eerste klik buiten een gefocuste cel verdween.
Worteloorzaak: table_embed_binding zet controller.skipRequestKeyboard = true als eenmalige bewaker bij cel-focus. Vroeg Quill daarna nooit om het
toetsenbord, dan bleef de vlag liggen en slikte hij de focusaanvraag van de
eerstvolgende buiten-klik.
Fix: een onTapDown buiten elke geregistreerde tabel laat de verlopen
bewaker vervallen, zodat de eerste klik al de caret plaatst.
Regressietest met echte muis-down/up en frames ertussen
(document_table_single_click_test.dart): beide richtingen, caret-positie,
direct typen, cel-focus, scrollstand en werkbalkgedrag.

Testplan

make check volledig groen (format, analyze, conventies, methodenlengte,
volledige testsuite, coverage en goldens)
Nieuwe tests falen zonder de fix (geverifieerd door tijdelijke revert)
Alle 31 bestaande tabel-tests groen

Generated with Devin

## Samenvatting - #2461: één klik tussen tabelcel en aangrenzende tekst werkte niet — de eerste klik buiten een gefocuste cel verdween. - Worteloorzaak: `table_embed_binding` zet `controller.skipRequestKeyboard = true` als eenmalige bewaker bij cel-focus. Vroeg Quill daarna nooit om het toetsenbord, dan bleef de vlag liggen en slikte hij de focusaanvraag van de eerstvolgende buiten-klik. - Fix: een `onTapDown` buiten elke geregistreerde tabel laat de verlopen bewaker vervallen, zodat de eerste klik al de caret plaatst. - Regressietest met echte muis-down/up en frames ertussen (`document_table_single_click_test.dart`): beide richtingen, caret-positie, direct typen, cel-focus, scrollstand en werkbalkgedrag. ## Testplan - [x] `make check` volledig groen (format, analyze, conventies, methodenlengte, volledige testsuite, coverage en goldens) - [x] Nieuwe tests falen zonder de fix (geverifieerd door tijdelijke revert) - [x] Alle 31 bestaande tabel-tests groen Generated with [Devin](https://devin.ai)

brenno added 1 commit

2026-10-19 17:26:17 +00:00

fix(document): één klik wisselt weer tussen tabel en tekst 56fb049da3

Zodra een tabelcel focus had, zette table_embed_binding een eenmalige
skipRequestKeyboard-bewaker. Vroeg Quill daarna nooit om het toetsenbord,
dan bleef de vlag liggen en consumeerde de eerstvolgende requestKeyboard —
precies de eerste klik buiten de tabel. Een tik buiten elke tabel laat de
verlopen bewaker nu vervallen (#2461).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno merged commit ae08be4b23 into main

2026-10-19 17:27:10 +00:00

brenno deleted branch fix/table-een-klik

2026-10-19 17:27:11 +00:00

brenno referenced this pull request from a commit

2026-10-19 17:27:12 +00:00

Merge pull request 'fix(document): één klik wisselt weer tussen tabel en tekst (#2461)' (#2482) from fix/table-een-klik into main