fix(opslaan): sla het bestemmingsvenster over als het niets te kiezen heeft (#646) #726

Merged

brenno merged 1 commit from fix/opslagdialoog-zonder-keuze-646 into main

2026-07-23 11:10:30 +00:00

brenno commented

2026-07-23 11:10:25 +00:00

Owner

Het vijfde item van #646: "Saving and opening pass through an extra intermediate dialog offering one real choice — three dialogs deep to save a file."

De klacht klopt op precies één pad, en dat is het pad waar iedereen begint. Het bestemmingsvenster laat je een bibliotheek kiezen; bij een eerste start ís er geen bibliotheek, dus zag je een lege lijst met twee knoppen ("Andere map…" en "Kies bestandsnaam…") die allebei naar hetzelfde systeemvenster leidden.

Het venster blijft. Mét bibliotheken doet het waar het voor bedoeld is: je kiest er een en ziet vooraf waar de presentatie, de afbeeldingen en de media terechtkomen. Het wordt overgeslagen op het ene pad waar het niets te bieden had — niet weggehaald.

Het beslismoment staat nu als shouldAskDestination los van de schil: een pure functie van drie vragen (nieuw deck, schrijfbaar bestandssysteem, bibliotheken). Die ingang had tot nu toe geen enkele test; nu vier, en de mutant hasLibraries → true laat precies de juiste rood worden.

make check groen: 5938 tests.

Het vijfde item van #646: "Saving and opening pass through an extra intermediate dialog offering one real choice — three dialogs deep to save a file." De klacht klopt op precies één pad, en dat is het pad waar iedereen begint. Het bestemmingsvenster laat je een bibliotheek kiezen; bij een eerste start ís er geen bibliotheek, dus zag je een lege lijst met twee knoppen ("Andere map…" en "Kies bestandsnaam…") die allebei naar hetzelfde systeemvenster leidden. **Het venster blijft.** Mét bibliotheken doet het waar het voor bedoeld is: je kiest er een en ziet vooraf waar de presentatie, de afbeeldingen en de media terechtkomen. Het wordt overgeslagen op het ene pad waar het niets te bieden had — niet weggehaald. Het beslismoment staat nu als `shouldAskDestination` los van de schil: een pure functie van drie vragen (nieuw deck, schrijfbaar bestandssysteem, bibliotheken). Die ingang had tot nu toe geen enkele test; nu vier, en de mutant `hasLibraries → true` laat precies de juiste rood worden. `make check` groen: 5938 tests.

brenno added 1 commit

2026-07-23 11:10:25 +00:00

fix(opslaan): sla het bestemmingsvenster over als het niets te kiezen heeft 0ec2a196df

Een nieuwe presentatie opslaan liep langs een venster dat je een
bibliotheek liet kiezen. Bij een eerste start ís er geen bibliotheek, dus
zag je een lege lijst met twee knoppen — "Andere map…" en "Kies
bestandsnaam…" — die allebei naar hetzelfde systeemvenster leidden. Dat
is de "drie dialogen diep om een bestand op te slaan" uit #646, en het
trof iedereen, want iedereen begint daar.

Het venster blijft. Mét bibliotheken doet het waar het voor bedoeld is:
je kiest er een en ziet vooraf waar de presentatie, de afbeeldingen en de
media terechtkomen. Het wordt overgeslagen op het ene pad waar het niets
te bieden had.

Het beslismoment staat nu als `shouldAskDestination` los van de schil —
een pure functie van drie vragen (nieuw deck, schrijfbaar bestandssysteem,
bibliotheken). Daarmee is het te toetsen zonder providers, notifier en
bestandskiezer op te tuigen; die ingang had tot nu toe geen enkele test.
Vier toetsen, en `hasLibraries → true` laat precies de juiste rood worden.

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

brenno merged commit 8d69d29457 into main

2026-07-23 11:10:30 +00:00

brenno deleted branch fix/opslagdialoog-zonder-keuze-646

2026-07-23 11:10:30 +00:00

brenno referenced this pull request from a commit

2026-07-23 11:10:32 +00:00

Merge pull request 'fix(opslaan): sla het bestemmingsvenster over als het niets te kiezen heeft (#646)' (#726) from fix/opslagdialoog-zonder-keuze-646 into main

brenno referenced this pull request

2026-07-23 11:25:45 +00:00

[Bug] The Settings sidebar truncates at the 200% text scale the app itself offers #646