fix: verberg map-kiezers op web (browser heeft geen bestandssysteem) #148

Merged

brenno merged 1 commit from fix/web-hide-native-folder-pickers into main 2026-07-08 17:43:48 +00:00

Conversation 0 Commits 1 Files changed 4 +98 -78

brenno commented 2026-07-08 17:28:48 +00:00

Owner

Copy link

Wat & waarom

Een web-gebruiker (Firefox) meldde dat de gele Kiezen-knop bij Presentatiemap en Exportmap niets deed — geen dialoog, geen console-fout.

Oorzaak: die knoppen roepen FilePicker.getDirectoryPath() aan. Die native map-kiezer is niet geïmplementeerd in de browser en geeft stil null terug, waarna de handler niets doet. Op web is er sowieso geen bestandssysteem — decks openen via bytes en elke export is een browser-download — dus een standaard presentatie-/exportmap heeft er geen betekenis.

Tijdens het onderzoek bleek dat Slide zoeken en Slides importeren in de slide-lijst exact dezelfde ongegate aanroep hebben (die functies scannen lokale mappen). De Openen-flow was al wél netjes afgeschermd op web.

Wijziging

Alle drie de controls worden nu op web verborgen via de bestaande supportsLocalProjectFolders-vlag (false op web), hetzelfde patroon als het welkomstscherm al gebruikt. Op desktop verandert er niets.

• lib/widgets/dialogs/parts/settings_dialog_general.dart — Presentatiemap + Exportmap-secties gegate
• lib/widgets/dialogs/settings_dialog.dart — import van platform_features
• lib/widgets/panels/slide_list_panel.dart — Slide zoeken/importeren-knoppen gegate
• CHANGELOG.md — één Fixed-entry

Verificatie

flutter analyze schoon op alle gewijzigde bestanden; dart format ongewijzigd. De logica is triviaal — supportsLocalProjectFolders is hard false in platform_features_web.dart.

🤖 Generated with Claude Code

## Wat & waarom Een web-gebruiker (Firefox) meldde dat de gele **Kiezen**-knop bij *Presentatiemap* en *Exportmap* niets deed — geen dialoog, geen console-fout. **Oorzaak:** die knoppen roepen `FilePicker.getDirectoryPath()` aan. Die native map-kiezer is niet geïmplementeerd in de browser en geeft stil `null` terug, waarna de handler niets doet. Op web is er sowieso geen bestandssysteem — decks openen via bytes en elke export is een browser-download — dus een standaard presentatie-/exportmap heeft er geen betekenis. Tijdens het onderzoek bleek dat **Slide zoeken** en **Slides importeren** in de slide-lijst exact dezelfde ongegate aanroep hebben (die functies scannen lokale mappen). De *Openen*-flow was al wél netjes afgeschermd op web. ## Wijziging Alle drie de controls worden nu op web verborgen via de bestaande `supportsLocalProjectFolders`-vlag (`false` op web), hetzelfde patroon als het welkomstscherm al gebruikt. Op desktop verandert er niets. - `lib/widgets/dialogs/parts/settings_dialog_general.dart` — Presentatiemap + Exportmap-secties gegate - `lib/widgets/dialogs/settings_dialog.dart` — import van `platform_features` - `lib/widgets/panels/slide_list_panel.dart` — Slide zoeken/importeren-knoppen gegate - `CHANGELOG.md` — één *Fixed*-entry ## Verificatie `flutter analyze` schoon op alle gewijzigde bestanden; `dart format` ongewijzigd. De logica is triviaal — `supportsLocalProjectFolders` is hard `false` in `platform_features_web.dart`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit 2026-07-08 17:28:48 +00:00

fix: verberg map-kiezers op web (browser heeft geen bestandssysteem) ... 4fa92fa8af

De "Kiezen"-knop bij Presentatiemap en Exportmap (Instellingen → Algemeen)
en de knoppen "Slide zoeken"/"Slides importeren" in de slide-lijst riepen
FilePicker.getDirectoryPath() aan. Die native map-kiezer bestaat niet in de
browser: de aanroep geeft stil null terug, dus klikken deed niets (geen fout,
geen dialoog). Op web is er sowieso geen bestandssysteem — decks openen via
bytes en elke export is een download — dus een mapkeuze heeft er geen betekenis.

Beide plekken worden nu op web verborgen via de bestaande
supportsLocalProjectFolders-vlag (false op web), zoals het welkomstscherm al
doet. Op desktop verandert er niets.

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

brenno merged commit ad126f78e0 into main 2026-07-08 17:43:48 +00:00

brenno deleted branch fix/web-hide-native-folder-pickers 2026-07-08 17:43:48 +00:00

brenno referenced this pull request from a commit 2026-07-08 17:43:50 +00:00

Merge pull request 'fix: verberg map-kiezers op web (browser heeft geen bestandssysteem)' (#148) from fix/web-hide-native-folder-pickers into main

brenno referenced this pull request 2026-07-22 08:47:41 +00:00

[Feature] Guard platform-gated file pickers so #150 cannot regress #505

brenno referenced this pull request 2026-07-22 08:47:42 +00:00

[Bug] Logo picker is not platform-gated and yields an unusable path on web #506

brenno referenced this pull request 2026-07-22 10:25:46 +00:00

fix: verberg de logokiezer op web in plaats van een pad te bewaren dat niet bestaat #525

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.

  

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

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!148

No description provided.