Use the new NLDD media-control icons in the video player modal

robbertbos commented

2026-07-24 05:29:00 +00:00

Owner

NLDD 0.8.70 adds play/pause/stop/forward/backward icons (plus filled variants). The video preview modal currently relies on native video controls only; if we add custom controls or an inline play affordance on video attachment cards, use these icons via cardIcons.ts. Low priority - only relevant when we touch the player.

robbertbos added the

nldd ds

label

2026-07-24 06:01:03 +00:00

robbertbos referenced this issue from a pull request that will close it,

2026-08-06 19:49:58 +00:00

NLDD icon buttons, and an icon per concept that exists and means it #245

robbertbos commented

2026-08-06 19:51:00 +00:00

Author

Owner

Opgepakt in #245, maar anders dan hier voorgesteld.

Bij het uitzoeken bleek de aanleiding scherper dan verwacht: AttachmentPreview mapte een video-bijlage op media, en die naam bestaat niet meer. NLDD gaf de media-transport-set een media--prefix toen die icoonbatch landde (forward was sowieso onbereikbaar, overschaduwd door de alias forward -> chevron-right). Elke video-bijlagekaart toonde dus een leeg vakje. Dat is nu play, wat ook is wat de kaart zelf al belooft ("... afspelen").

Geen eigen transportcontrols. De speler houdt <video controls>: zelfgebouwde knoppen betekenen het toetsenbord-, scrub- en screenreader-gedrag opnieuw bouwen dat de browser al goed doet, en NLDD's eigen richtlijn is om op native HTML te leunen.

Eén van de iconen uit deze issue kreeg wel een tweede plek: media-forward-end staat nu op "Verzend nu" bij uitgaande post, waar een terugdraaiende klok (alias history) stond terwijl de actie juist het wachten overslaat.

En het duurzame stuk: iconNames.test.ts faalt voortaan op een icoonnaam die noch in de registry noch in de aliassen staat. Zonder die test verdwijnt zo'n icoon geruisloos - geen waarschuwing, geen fout, en typecheck ziet het niet omdat het veld string is.

Opgepakt in #245, maar anders dan hier voorgesteld. Bij het uitzoeken bleek de aanleiding scherper dan verwacht: `AttachmentPreview` mapte een video-bijlage op `media`, en die naam bestaat niet meer. NLDD gaf de media-transport-set een `media-`-prefix toen die icoonbatch landde (`forward` was sowieso onbereikbaar, overschaduwd door de alias `forward` -> `chevron-right`). Elke video-bijlagekaart toonde dus een leeg vakje. Dat is nu `play`, wat ook is wat de kaart zelf al belooft ("... afspelen"). Geen eigen transportcontrols. De speler houdt `<video controls>`: zelfgebouwde knoppen betekenen het toetsenbord-, scrub- en screenreader-gedrag opnieuw bouwen dat de browser al goed doet, en NLDD's eigen richtlijn is om op native HTML te leunen. Eén van de iconen uit deze issue kreeg wel een tweede plek: `media-forward-end` staat nu op "Verzend nu" bij uitgaande post, waar een terugdraaiende klok (alias `history`) stond terwijl de actie juist het wachten overslaat. En het duurzame stuk: `iconNames.test.ts` faalt voortaan op een icoonnaam die noch in de registry noch in de aliassen staat. Zonder die test verdwijnt zo'n icoon geruisloos - geen waarschuwing, geen fout, en typecheck ziet het niet omdat het veld `string` is.

robbertbos closed this issue

2026-08-06 20:04:36 +00:00

Rows
Columns

Use the new NLDD media-control icons in the video player modal #171