Meerdere bestanden tegelijk openen kan niet #1928

Closed
opened 2026-09-02 18:32:44 +00:00 by brenno · 3 comments
Owner

Wat er gebeurt

Bij 'Presentatie openen' (Ctrl/Cmd+O) kun je precies één bestand aanwijzen. Wie drie presentaties of documenten wil openen, moet drie keer de hele dialoog doorlopen. Dat geldt zowel voor de lijst in de OciDeck-dialoog als voor de native bestandskiezer achter 'Bladeren…'.

Wat er hoort te gebeuren

Meerdere bestanden aanklikken en in één keer openen — elk in een eigen tabblad. OciDeck heeft al tabbladen, dus het gedrag past bij het programma; sleep-en-neerzetten van meerdere bestanden werkt zelfs al wél (_onFilesDropped in lib/widgets/app_shell.dart verwerkt een lijst). De openen-dialoog blijft daarbij achter.

Verwacht gedrag in detail:

  • In de lijst van de openen-dialoog: meerdere rijen selecteren met Ctrl/Cmd+klik en Shift+klik (aaneengesloten bereik).
  • Achter 'Bladeren…': de native kiezer met allowMultiple.
  • Openen zet elk bestand in een eigen tabblad; het laatst geopende wordt actief.
  • Springen naar een zoektreffer (slide-index) blijft alleen gelden bij een enkele selectie — bij meerdere bestanden is er geen zinnige index.
  • Fouten per bestand: één onleesbaar bestand mag de rest niet afbreken.

Raakvlakken

  • lib/widgets/dialogs/open_presentation_dialog.dartOpenSearchResult levert nu één pad.
  • lib/widgets/shell/shell_actions.dart_openWithSearch / _openWithBytesPicker.
  • lib/services/file_service.dartpickMarkdownFile (en de web-variant pickDeckFileBytes).
## Wat er gebeurt Bij 'Presentatie openen' (Ctrl/Cmd+O) kun je precies één bestand aanwijzen. Wie drie presentaties of documenten wil openen, moet drie keer de hele dialoog doorlopen. Dat geldt zowel voor de lijst in de OciDeck-dialoog als voor de native bestandskiezer achter 'Bladeren…'. ## Wat er hoort te gebeuren Meerdere bestanden aanklikken en in één keer openen — elk in een eigen tabblad. OciDeck heeft al tabbladen, dus het gedrag past bij het programma; sleep-en-neerzetten van meerdere bestanden werkt zelfs al wél (`_onFilesDropped` in `lib/widgets/app_shell.dart` verwerkt een lijst). De openen-dialoog blijft daarbij achter. Verwacht gedrag in detail: - In de lijst van de openen-dialoog: meerdere rijen selecteren met Ctrl/Cmd+klik en Shift+klik (aaneengesloten bereik). - Achter 'Bladeren…': de native kiezer met `allowMultiple`. - Openen zet elk bestand in een eigen tabblad; het laatst geopende wordt actief. - Springen naar een zoektreffer (slide-index) blijft alleen gelden bij een enkele selectie — bij meerdere bestanden is er geen zinnige index. - Fouten per bestand: één onleesbaar bestand mag de rest niet afbreken. ## Raakvlakken - `lib/widgets/dialogs/open_presentation_dialog.dart` — `OpenSearchResult` levert nu één pad. - `lib/widgets/shell/shell_actions.dart` — `_openWithSearch` / `_openWithBytesPicker`. - `lib/services/file_service.dart` — `pickMarkdownFile` (en de web-variant `pickDeckFileBytes`).
Author
Owner

Opgepakt. Tak: feat/open-multiple-files.

Verwachte reikwijdte: OpenSearchResult naar meerdere paden, meervoudige selectie (Ctrl/Cmd+klik en Shift+klik) in open_presentation_dialog.dart, allowMultiple in de native kiezer (file_service.dart), en per-bestand foutafhandeling in _openWithSearch. Wordt gebouwd ná #1927, omdat die dezelfde openroute raakt.

Opgepakt. Tak: `feat/open-multiple-files`. Verwachte reikwijdte: `OpenSearchResult` naar meerdere paden, meervoudige selectie (Ctrl/Cmd+klik en Shift+klik) in `open_presentation_dialog.dart`, `allowMultiple` in de native kiezer (`file_service.dart`), en per-bestand foutafhandeling in `_openWithSearch`. Wordt gebouwd ná #1927, omdat die dezelfde openroute raakt.
Author
Owner

PR: #1935 (tak feat/open-multiple-files, acht commits).

Shift = bereik, Ctrl/Cmd = toevoegen/verwijderen, kale klik opent zoals altijd meteen dat ene bestand; "Openen (n)" opent de hele selectie, elk in een eigen tabblad. Achter "Bladeren…" neemt ook de systeemkiezer een meervoudige selectie aan (op macOS via het eigen NSOpenPanel).

make check groen, scans schoon, l10n voor 2 nieuwe strings in alle 31 talen. Elke test is met een mutatie geproefd, en het scherm mét selectie staat nu in overflow_stress_test (drie knoppen in de voet bij 200% tekst).

Twee dingen die openstaan:

  • De beeldkeuring in de draaiende app is niet gedaan — de toegangsvraag werd geweigerd. Mechanisch getoetst zijn de overflow en het zichtbare verschil tussen een aangewezen en een gewone rij; hoe het er in licht en donker uitziet is niet met eigen ogen gezien.
  • De bewaker-toets bracht één bevinding op: de bestandslijst is niet met het toetsenbord te bedienen. Bestaand gedrag, niet door deze wijziging veroorzaakt, apart ingediend als #1934.
PR: https://pawprint.vigilis.online/LibreKAT/Ocideck/pulls/1935 (tak `feat/open-multiple-files`, acht commits). Shift = bereik, Ctrl/Cmd = toevoegen/verwijderen, kale klik opent zoals altijd meteen dat ene bestand; "Openen (n)" opent de hele selectie, elk in een eigen tabblad. Achter "Bladeren…" neemt ook de systeemkiezer een meervoudige selectie aan (op macOS via het eigen `NSOpenPanel`). `make check` groen, scans schoon, l10n voor 2 nieuwe strings in alle 31 talen. Elke test is met een mutatie geproefd, en het scherm mét selectie staat nu in `overflow_stress_test` (drie knoppen in de voet bij 200% tekst). Twee dingen die openstaan: - De beeldkeuring in de draaiende app is **niet** gedaan — de toegangsvraag werd geweigerd. Mechanisch getoetst zijn de overflow en het zichtbare verschil tussen een aangewezen en een gewone rij; hoe het er in licht en donker uitziet is niet met eigen ogen gezien. - De bewaker-toets bracht één bevinding op: de bestandslijst is niet met het toetsenbord te bedienen. Bestaand gedrag, niet door deze wijziging veroorzaakt, apart ingediend als #1934.
Author
Owner

Gemerged in main als b09521327 (PR #1935).

Shift = bereik, Ctrl/Cmd = toevoegen/verwijderen, kale klik opent zoals altijd meteen dat ene bestand; "Openen (n)" opent de hele selectie, elk in een eigen tabblad. Achter "Bladeren…" neemt ook de systeemkiezer een meervoudige selectie aan.

Wat er níet in zat, en waarom:

  • Beeldkeuring in de draaiende app. De toegangsvraag voor computer-use werd geweigerd, dus de dialoog is niet met eigen ogen gezien. Mechanisch getoetst: geen overflow bij 200% tekst over vier viewports (het scherm mét selectie staat nu in overflow_stress_test) en het zichtbare verschil tussen een aangewezen en een gewone rij. De kleuren komen uit thema-tokens, geen eigen palet.
  • Toetsenbordbediening van de lijst. Bestaand gedrag, niet door deze wijziging veroorzaakt; apart als #1934.
Gemerged in main als `b09521327` (PR #1935). Shift = bereik, Ctrl/Cmd = toevoegen/verwijderen, kale klik opent zoals altijd meteen dat ene bestand; "Openen (n)" opent de hele selectie, elk in een eigen tabblad. Achter "Bladeren…" neemt ook de systeemkiezer een meervoudige selectie aan. Wat er níet in zat, en waarom: - **Beeldkeuring in de draaiende app.** De toegangsvraag voor computer-use werd geweigerd, dus de dialoog is niet met eigen ogen gezien. Mechanisch getoetst: geen overflow bij 200% tekst over vier viewports (het scherm mét selectie staat nu in `overflow_stress_test`) en het zichtbare verschil tussen een aangewezen en een gewone rij. De kleuren komen uit thema-tokens, geen eigen palet. - **Toetsenbordbediening van de lijst.** Bestaand gedrag, niet door deze wijziging veroorzaakt; apart als #1934.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
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#1928
No description provided.