LibreKAT/Ocideck
LibreKAT/Ocideck
Watch
4
0
Fork
You've already forked Ocideck
 1
Code Issues Pull requests Projects Releases 21 Packages Activity Actions

Resizable dialogen waar lange paden afgekapt worden (voortbouw op #1211) #1217

New issue
Closed
opened 2026-08-04 14:12:17 +00:00 by brenno · 1 comment
brenno commented 2026-08-04 14:12:17 +00:00
Owner
Copy link

Voortbouwend op #1211 (save-dialoog: paden volledig tonen + breedte-aanpasbaar). Tijdens dat werk bekeek ik waar in de UI nog lange bestandspaden met TextOverflow.ellipsis afgekapt worden.

Bevindingen

Hoogste prioriteit (tonen paden, vaste breedte, geen of beperkte tooltip):

  • s3_browser_dialog.dart — breadcrumb /pad/naar/bucket, width: 560, ellipsis, geen tooltip
  • webdav_browser_dialog.dart — breadcrumb /pad/naar/webdav, width: 560, ellipsis, geen tooltip

Gemiddelde prioriteit (lijstweergaven met pad per item, al compact via displayFolder() + tooltip, maar vaste breedte):

  • scan_library_dialog.dart — width: 760
  • duplicate_cleanup_dialog.dart — width: 640
  • open_presentation_dialog.dart — pad + bestandsnaam + slides

Geen actie nodig (tonen namen/titels, geen paden): slide_finder, asset_usage, tab_bar, status_bar, storage_connection_picker, git_browser, presentation_info, seal_timestamp, export, import_slides.

Oplossing

  • softWrap is niet geschikt voor breadcrumbs (wrapt de header) of lijsten (wordt onbruikbaar lang). De juiste oplossing is resizable dialogen, zoals #1211 voor de save-dialoog introduceerde.
  • Omdat de resize-logica (state + drag-handle + clamp) 5× herhaald zou worden, bouwen we een gedeelde resizable-Dialog-helper en passen die toe op de 5 dialogen.
  • De 2 browsers krijgen ook een Tooltip op de breadcrumb (goedkoopste winst).
  • De save-dialoog uit #1211 wordt niet gemigreerd naar de helper in deze PR (werkt al); eventueel later.

Marp / file-format impact
No

Additional context
Voortbouw op #1211 / PR #1216.

Voortbouwend op #1211 (save-dialoog: paden volledig tonen + breedte-aanpasbaar). Tijdens dat werk bekeek ik waar in de UI nog lange bestandspaden met `TextOverflow.ellipsis` afgekapt worden. **Bevindingen** Hoogste prioriteit (tonen paden, vaste breedte, geen of beperkte tooltip): - `s3_browser_dialog.dart` — breadcrumb `/pad/naar/bucket`, `width: 560`, `ellipsis`, geen tooltip - `webdav_browser_dialog.dart` — breadcrumb `/pad/naar/webdav`, `width: 560`, `ellipsis`, geen tooltip Gemiddelde prioriteit (lijstweergaven met pad per item, al compact via `displayFolder()` + tooltip, maar vaste breedte): - `scan_library_dialog.dart` — `width: 760` - `duplicate_cleanup_dialog.dart` — `width: 640` - `open_presentation_dialog.dart` — pad + bestandsnaam + slides **Geen actie nodig** (tonen namen/titels, geen paden): slide_finder, asset_usage, tab_bar, status_bar, storage_connection_picker, git_browser, presentation_info, seal_timestamp, export, import_slides. **Oplossing** - softWrap is niet geschikt voor breadcrumbs (wrapt de header) of lijsten (wordt onbruikbaar lang). De juiste oplossing is **resizable dialogen**, zoals #1211 voor de save-dialoog introduceerde. - Omdat de resize-logica (state + drag-handle + clamp) 5× herhaald zou worden, bouwen we een **gedeelde resizable-Dialog-helper** en passen die toe op de 5 dialogen. - De 2 browsers krijgen ook een `Tooltip` op de breadcrumb (goedkoopste winst). - De save-dialoog uit #1211 wordt niet gemigreerd naar de helper in deze PR (werkt al); eventueel later. **Marp / file-format impact** No **Additional context** Voortbouw op #1211 / PR #1216.
brenno commented 2026-08-04 14:35:14 +00:00
Author
Owner
Copy link

Opgelost in PR #1218. Vijf dialogen zijn nu breedte-aanpasbaar via een gedeelde ResizableDialogBox-helper:

  • S3-browser en WebDAV-browser: resizable + tooltip op de breadcrumb
  • Presentaties zoeken, dubbele presentaties opruimen, presentatie openen: resizable

softWrap is bewust niet toegepast op deze 5 dialogen: een breadcrumb die wrapt breekt de navigatie-header, en in een lijst maakt softWrap elke regel 2-3x zo hoog. De gedeelde helper gebruikt een builder-patroon dat de sleephandgreep levert; de aanroeler plaatst hem waar het past. De save-dialoog uit #1211 wordt in deze PR niet gemigreerd (werkt al). 5 nieuwe tests voor de helper, bestaande dialog-tests groen, l10n in alle 31 talen, source map bijgewerkt.

Opgelost in PR #1218. Vijf dialogen zijn nu breedte-aanpasbaar via een gedeelde `ResizableDialogBox`-helper: - S3-browser en WebDAV-browser: resizable + tooltip op de breadcrumb - Presentaties zoeken, dubbele presentaties opruimen, presentatie openen: resizable softWrap is bewust niet toegepast op deze 5 dialogen: een breadcrumb die wrapt breekt de navigatie-header, en in een lijst maakt softWrap elke regel 2-3x zo hoog. De gedeelde helper gebruikt een builder-patroon dat de sleephandgreep levert; de aanroeler plaatst hem waar het past. De save-dialoog uit #1211 wordt in deze PR niet gemigreerd (werkt al). 5 nieuwe tests voor de helper, bestaande dialog-tests groen, l10n in alle 31 talen, source map bijgewerkt.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
v0.5.0
v0.4.10
v0.4.9
v0.4.8
v0.4.7
v0.4.6
v0.4.5
v0.4.4
v0.4.3
v0.4.2
v0.4.1
v0.4.0
v0.3.6
v0.3.5
v0.3.4
v0.3.3
v0.3.2
v0.3.1
v0.3.0
v0.2.0
v0.1.3
v0.1.2
v0.1.1
v0.1.0
archive/git-mirror
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
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#1217
No description provided.