fix(import): de presentatie-import was op web onbereikbaar (#772) #838

Merged

brenno merged 2 commits from fix/import-module-web-772 into main

2026-07-25 00:00:24 +00:00

brenno commented

2026-07-24 23:57:40 +00:00

Owner

Gevonden bij de laatste controle vóór de eerste v0.1.0-tag.

Wat er mis was

De presentatie-import (PowerPoint/Keynote/Impress, #772) is op web onbereikbaar, terwijl hij daar juist hoort te werken: hij draait op bytes, zonder dart:io, en de gebruikershandleiding belooft de webversie ook al. De module Importeren stond op web echter helemaal uit, met "de mapkiezer bestaat hier niet" als reden — die reden gold OpenKAT (dat een map van schijf leest) en is nooit herzien toen de presentatie-import als tweede bron in dezelfde module kwam. De schakelaar die de bron onthult was op web dus dood.

De fix

De schakelaar werkt nu overal. Wat op web wegvalt is alleen de OpenKAT-helft:

Modulekaart — de schakelaar is niet meer op web uitgeschakeld; de tekst is platform-bewust (op web zonder de OpenKAT-zin) en noemt nu eindelijk béíde bronnen, en de OpenKAT-voettekst (rapportagemap + Naar Integraties) blijft desktop-only.
SettingsSection.navItems — het tabblad Integraties is volledig OpenKAT en blijft op web verborgen (openKatAvailable), zodat er geen leeg tabblad ontstaat. De parameter is puur en testbaar.
Instellingenzoeker — de OpenKAT-entry krijgt integrationsOnly: "openkat" leidt niet meer naar een tabblad dat er niet is (op web, of met de module uit).

Toetsing

make check groen in één pass — 6596 tests, 86,5% dekking.
Twee tests op de tab-gating, waaronder de web-regressie (Integraties valt weg ook met de module aan).
31-talen-l10n voor de herschreven kaarttekst; twee verweesde strings verwijderd. make l10n-check groen.
Docs: SOURCE_MAP-claim herschreven (stond nog op "op web uitgeschakeld"), CHANGELOG-ontwikkellogregel. USER_GUIDE bleek al correct — die beloofde de web-import al.

Let op — releasebeslissing

Dit verandert wat de publieke webdemo (ocideck.librekat.nl) in v0.1.0 aanbiedt: bezoekers kunnen straks hun presentaties in de browser openzetten (in het geheugen, met dezelfde security-gate als desktop). De richting is bevestigd; de merge/tag wacht op akkoord.

🤖 Generated with Claude Code

Gevonden bij de laatste controle vóór de eerste `v0.1.0`-tag. ## Wat er mis was De presentatie-import (PowerPoint/Keynote/Impress, #772) is **op web onbereikbaar**, terwijl hij daar juist hoort te werken: hij draait op bytes, zonder `dart:io`, en de gebruikershandleiding belooft de webversie ook al. De module *Importeren* stond op web echter helemaal uit, met "de mapkiezer bestaat hier niet" als reden — die reden gold **OpenKAT** (dat een map van schijf leest) en is nooit herzien toen de presentatie-import als tweede bron in dezelfde module kwam. De schakelaar die de bron onthult was op web dus dood. ## De fix De schakelaar werkt nu overal. Wat op web wegvalt is alleen de OpenKAT-helft: - **Modulekaart** — de schakelaar is niet meer op web uitgeschakeld; de tekst is platform-bewust (op web zonder de OpenKAT-zin) en noemt nu eindelijk béíde bronnen, en de OpenKAT-voettekst (rapportagemap + *Naar Integraties*) blijft desktop-only. - **`SettingsSection.navItems`** — het tabblad *Integraties* is volledig OpenKAT en blijft op web verborgen (`openKatAvailable`), zodat er geen leeg tabblad ontstaat. De parameter is puur en testbaar. - **Instellingenzoeker** — de OpenKAT-entry krijgt `integrationsOnly`: "openkat" leidt niet meer naar een tabblad dat er niet is (op web, of met de module uit). ## Toetsing - `make check` groen in één pass — 6596 tests, 86,5% dekking. - Twee tests op de tab-gating, waaronder de web-regressie (Integraties valt weg ook met de module aan). - 31-talen-l10n voor de herschreven kaarttekst; twee verweesde strings verwijderd. `make l10n-check` groen. - Docs: SOURCE_MAP-claim herschreven (stond nog op "op web uitgeschakeld"), CHANGELOG-ontwikkellogregel. USER_GUIDE bleek al correct — die beloofde de web-import al. ## Let op — releasebeslissing Dit **verandert wat de publieke webdemo (`ocideck.librekat.nl`) in `v0.1.0` aanbiedt**: bezoekers kunnen straks hun presentaties in de browser openzetten (in het geheugen, met dezelfde security-gate als desktop). De richting is bevestigd; de merge/tag wacht op akkoord. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 2 commits

2026-07-24 23:57:41 +00:00

fix(import): de presentatie-import was op web onbereikbaar (#772 ) b1f66902d7

De module Importeren stond op web helemaal uit, met "de mapkiezer bestaat hier
niet" als reden — die reden gold OpenKAT, dat een map van schijf leest, en is
nooit herzien toen de presentatie-import (#772) als tweede bron in dezelfde
module kwam. Die import draait op bytes (geen dart:io) en werkt dus wél in de
browser; de gebruikershandleiding beloofde dat al, maar de schakelaar die de
bron onthult was op web dood.

De schakelaar werkt nu overal. Wat op web wegvalt is alleen de OpenKAT-helft: de
kaarttekst laat daar de OpenKAT-zin weg, de voettekst met de rapportagemap
verdwijnt, en het tabblad Integraties (volledig OpenKAT) blijft op web verborgen
via SettingsSection.navItems(openKatAvailable) in plaats van als leeg tabblad te
verschijnen. De instellingenzoeker verbergt de OpenKAT-entry (integrationsOnly)
waar dat tabblad niet bestaat. De kaarttekst noemt nu beide bronnen; hij sprak
alleen nog over OpenKAT.

Inclusief 31-talen-l10n voor de herschreven kaarttekst en twee tests op de
tab-gating (o.a. de web-regressie).

Gevonden bij de laatste controle vóór de eerste tag.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

docs(source-map): de web-bereikbaarheid van de presentatie-import (#772 )

scans / scans (pull_request) Successful in 3m23s

Details

067e7355d7

SOURCE_MAP beschreef de importmodulekaart nog als "op web uitgeschakeld — de
knop die liegt"; dat was precies het oude gedrag. Herschreven naar het nieuwe:
de schakelaar werkt op web, alleen de OpenKAT-helft valt daar weg. Plus een
CHANGELOG-ontwikkellogregel.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit 9b663bfd33 into main

2026-07-25 00:00:24 +00:00

brenno referenced this pull request from a commit

2026-07-25 00:00:26 +00:00

Merge pull request 'fix(import): de presentatie-import was op web onbereikbaar (#772)' (#838) from fix/import-module-web-772 into main