fix(document): documenttabblad sluit en noemt de soort (#1614) #1618

Merged
brenno merged 1 commit from fix/1614-doc-tab-close into main 2026-08-20 17:57:29 +00:00
Owner

Samenvatting

Een vuil documenttabblad sluiten gooide een StateError omdat requestCloseTab tab.deckNotifier aanriep — dat is null voor een document. De sluitbevestiging noemde bovendien "presentatie" voor elk tabblad, ook voor een document.

  • requestCloseTab routeert nu via saveTabWithDestination, die op soort schakelt: een document byte-getrouw via zijn eigen pad, een presentatie via de deck-route.
  • De vier meldingen (hersteldialoog, herstellijst, afsluiten, tabblad sluiten) noemen de soort: "document" of "presentatie", en in het meervoud "bestanden".
  • De herstellijst toont een pictogram per soort.
  • De meervoudsstring gebruikt een plaatshouder ({n}), niet drie stukken met het getal ertussen — dat legde de Nederlandse woordvolgorde op aan 31 talen.
  • 31 vertaalbestanden bijgewerkt; verweesde strings verwijderd.

Bewaker

Deze wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte. Het is een bugfix in de sluitroute en een terminologie-correctie in meldingen. De bewaker-stap is expliciet overgeslagen.

Testplan

  • make check groen (format, analyse, 10240 tests, dekking 87.2%, per-file vloer)
  • Regressietest: een vuil documenttabblad noemt "document", niet "presentatie"
  • Regressietest: een vuil documenttabblad sluit op Opslaan zonder StateError
  • make check-secrets niet gedraaid (geen nieuwe dependencies of sleutels)
  • make sast niet gedraaid (geen nieuwe dependencies of sleutels)

Generated with Devin

## Samenvatting Een vuil documenttabblad sluiten gooide een `StateError` omdat `requestCloseTab` `tab.deckNotifier` aanriep — dat is `null` voor een document. De sluitbevestiging noemde bovendien "presentatie" voor elk tabblad, ook voor een document. - `requestCloseTab` routeert nu via `saveTabWithDestination`, die op soort schakelt: een document byte-getrouw via zijn eigen pad, een presentatie via de deck-route. - De vier meldingen (hersteldialoog, herstellijst, afsluiten, tabblad sluiten) noemen de soort: "document" of "presentatie", en in het meervoud "bestanden". - De herstellijst toont een pictogram per soort. - De meervoudsstring gebruikt een plaatshouder (`{n}`), niet drie stukken met het getal ertussen — dat legde de Nederlandse woordvolgorde op aan 31 talen. - 31 vertaalbestanden bijgewerkt; verweesde strings verwijderd. ## Bewaker Deze wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte. Het is een bugfix in de sluitroute en een terminologie-correctie in meldingen. De bewaker-stap is expliciet overgeslagen. ## Testplan - [x] `make check` groen (format, analyse, 10240 tests, dekking 87.2%, per-file vloer) - [x] Regressietest: een vuil documenttabblad noemt "document", niet "presentatie" - [x] Regressietest: een vuil documenttabblad sluit op Opslaan zonder `StateError` - [x] `make check-secrets` niet gedraaid (geen nieuwe dependencies of sleutels) - [x] `make sast` niet gedraaid (geen nieuwe dependencies of sleutels) Generated with [Devin](https://devin.ai)
fix(document): documenttabblad sluit en noemt de soort (#1614)
All checks were successful
scans / scans (pull_request) Successful in 2m17s
static-gate / static-gate (pull_request) Successful in 5m9s
7ef703fcbf
Een vuil documenttabblad sluiten gooide een StateError omdat requestCloseTab
tab.deckNotifier aanriep — dat is null voor een document. De sluitbevestiging
noemde bovendien "presentatie" voor elk tabblad, ook voor een document.

requestCloseTab routeert nu via saveTabWithDestination, die op soort schakelt:
een document byte-getrouw via zijn eigen pad, een presentatie via de deck-route.
De vier meldingen (hersteldialoog, herstellijst, afsluiten, tabblad sluiten)
noemen de soort: "document" of "presentatie", en in het meervoud "bestanden".
De herstellijst toont een pictogram per soort. De meervoudsstring gebruikt een
plaatshouder ({n}), niet drie stukken met het getal ertussen.

Regressietests: een vuil documenttabblad noemt "document" niet "presentatie",
en sluit op Opslaan zonder StateError.

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

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
brenno merged commit 45affde1cf into main 2026-08-20 17:57:29 +00:00
Sign in to join this conversation.
No description provided.