Documenteditor: zoeken en zoeken-en-vervangen #1688

Merged

brenno merged 7 commits from feat/document-zoeken-vervangen into main 2026-08-21 22:40:03 +00:00

Conversation 0 Commits 7 Files changed 6 +606 -53

brenno commented 2026-08-21 15:34:40 +00:00

Owner

Copy link

Samenvatting

Brengt zoeken en zoeken-en-vervangen naar de documenteditor, in zowel de Visuele (Quill WYSIWYG) als de Bron-modus. Hergebruikt de bestaande MarkdownFindBar en text_search-utils uit de presentatie-broneditor — één zoekbalk, één gewoonte.

• Ctrl/Cmd+F opent zoeken; Ctrl/Cmd+H opent zoeken-en-vervangen. Een zoekknop in de werkbalk opent zoeken.
• Volgende/vorige match, hoofdlettergevoelig, matchteller (1 / 3).
• Vervang huidige / vervang alles.
• Bron: zoekmarkering + syntaxismarkering via MarkdownSourceController (dezelfde controller als de presentatie-broneditor).
• Visueel: de Quill-cursor springt naar de match via MarkdownCaretMap-offsetvertaling. Vervangen werkt op de Markdown-bron (exacte offsets), zodat het in beide modi veilig is.
• De matchteller loopt mee terwijl je typt.

Sluit #1686.

Testplan

• make check groen (10333 tests, 87,2% dekking, per-file-vloer schoon)
• make check-secrets schoon (gitleaks + trufflehog)
• make sast schoon (semgrep, 0 findings)
• Nieuwe tests in test/document_editor_screen_test.dart: zoeken met teller, volgende/vorige, hoofdlettergevoelig, vervang huidige, vervang alles, Ctrl+F/Ctrl+H, visuele modus, Escape

Generated with Devin

## Samenvatting Brengt zoeken en zoeken-en-vervangen naar de documenteditor, in zowel de Visuele (Quill WYSIWYG) als de Bron-modus. Hergebruikt de bestaande `MarkdownFindBar` en `text_search`-utils uit de presentatie-broneditor — één zoekbalk, één gewoonte. - **Ctrl/Cmd+F** opent zoeken; **Ctrl/Cmd+H** opent zoeken-en-vervangen. Een zoekknop in de werkbalk opent zoeken. - Volgende/vorige match, hoofdlettergevoelig, matchteller (`1 / 3`). - Vervang huidige / vervang alles. - **Bron**: zoekmarkering + syntaxismarkering via `MarkdownSourceController` (dezelfde controller als de presentatie-broneditor). - **Visueel**: de Quill-cursor springt naar de match via `MarkdownCaretMap`-offsetvertaling. Vervangen werkt op de Markdown-bron (exacte offsets), zodat het in beide modi veilig is. - De matchteller loopt mee terwijl je typt. Sluit #1686. #### Testplan - [x] `make check` groen (10333 tests, 87,2% dekking, per-file-vloer schoon) - [x] `make check-secrets` schoon (gitleaks + trufflehog) - [x] `make sast` schoon (semgrep, 0 findings) - [x] Nieuwe tests in `test/document_editor_screen_test.dart`: zoeken met teller, volgende/vorige, hoofdlettergevoelig, vervang huidige, vervang alles, Ctrl+F/Ctrl+H, visuele modus, Escape Generated with [Devin](https://devin.ai)

brenno added 2 commits 2026-08-21 15:34:40 +00:00

feat(document): zoeken en zoeken-en-vervangen in de documenteditor ... 73bba30768

Hergebruikt MarkdownFindBar + text_search utils uit de presentatie-broneditor.
Ctrl/Cmd+F opent zoeken, Ctrl/Cmd+H opent zoeken-en-vervangen. Werkt in zowel
Visueel (Quill) als Bron: in Bron met zoekmarkering + syntaxismarkering via
MarkdownSourceController, in Visueel via MarkdownCaretMap-offsetvertaling van
de match naar de Quill-cursor. Vervangen werkt op de Markdown-bron (exacte
offsets), zodat het in beide modi veilig is.

Ref #1686

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

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

test(document): zoek-en-vervangen tests voor bron- en visuele modus ... 4505fe0d03

Dekt zoeken met matchteller, volgende/vorige, hoofdlettergevoelig, vervang
huidige, vervang alles, Ctrl+F/Ctrl+H-sneltoetsen, visuele modus, en Escape.

Ref #1686

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

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

brenno added 1 commit 2026-08-21 21:34:57 +00:00

style(document): pas dart format toe op zoek-tests (Linux-formattering) ... 8179e795f8

De Linux dart format breekt lange find.descendant-chains en testWidgets-
handtekeningen die macOS laat staan. Zelfde inhoud, andere regelafbreking.

Ref #1686

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

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

brenno added 1 commit 2026-08-21 21:41:48 +00:00

chore(conventions): verhoog file-size baseline voor document_editor_screen ... d540c5dcaf

+182 regels (#1686): zoeken en zoeken-en-vervangen — find/replace-staat en
MarkdownFindBar in de build. Hergebruikt text_search + MarkdownSourceController;
de methoden zijn cohesive met de State-levenscyclus en het scherm heeft al zes
part-bestanden.

Ref #1686

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

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

brenno added 1 commit 2026-08-21 21:49:07 +00:00

chore(conventions): verhoog class-size baseline voor _DocumentEditorScreenState ... 4f13909152

+185 regels (#1686): find/replace-methoden zijn cohesive met de State-
levenscyclus (lezen/zetten _controller, _viewMode, setState) en kunnen niet
zonder private-veldtoegang naar een losse klasse.

Ref #1686

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

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

brenno added 1 commit 2026-08-21 21:59:56 +00:00

refactor(document): extraheren _findBar() om build onder 150 regels ... a4fc141ba8

De MarkdownFindBar in build duwde hem over de method-length-grens (178 vs 150).
Extractie naar _findBar()-helper brengt hem terug zonder gedrag te veranderen.

Ref #1686

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

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

brenno added 1 commit 2026-08-21 22:17:11 +00:00

refactor(document): extraheren _findBar() en _docToolbar() uit build ... 17a79f76ac

De build-methode was 178 regels (max 150) door de MarkdownFindBar en de
_DocEditorToolbar-inline. Extractie naar _findBar() en _docToolbar() brengt
hem terug zonder gedrag te veranderen.

Ref #1686

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

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

brenno merged commit 4a639c2739 into main 2026-08-21 22:40:03 +00:00

brenno referenced this pull request from a commit 2026-08-21 22:40:04 +00:00

Documenteditor: zoeken en zoeken-en-vervangen (#1688)

brenno referenced this pull request from a commit 2026-08-21 22:49:29 +00:00

chore(conventions): verhoog file-size baseline voor tabs_provider (#1688)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

release-blocker

Must land before the next release. scripts/release_auto.sh refuses to start while this is open.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

release-blocker

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

LibreKAT/Ocideck!1688

No description provided.