Composer: popover/menu-wiring via het popup-slot (NLDD 0.8.69) #131

New issue

Closed

opened 2026-07-19 20:57:43 +00:00 by robbertbos · 1 comment

robbertbos commented

2026-07-19 20:57:43 +00:00

Owner

NLDD 0.8.69 geeft nldd-button en nldd-icon-button een popup-slot: een geneste nldd-menu of nldd-popover wordt automatisch verankerd en getoggeld, inclusief expanded/aria-haspopup-sync.

FormatToolbar.vue doet dat nu allemaal met de hand voor het kopjes-menu: headingTriggerRef, headingMenuRef, headingOpen, menu.anchorElement = trigger en een eigen toggle-handler. Dat kan grotendeels weg.

Ook nalopen waar de trigger een nldd-button is (o.a. ReactionBar.vue). Niet van toepassing: MentionProfilePopover/ReactorPopover (anker is een willekeurig element in de tekst) en SendButton.vue (blijft bewust custom vanwege de primary-variant-styling van nldd-split-button, ongewijzigd in 0.8.69).

Afhankelijk van PR #124 (bump naar ^0.8.69).

NLDD 0.8.69 geeft `nldd-button` en `nldd-icon-button` een `popup`-slot: een geneste `nldd-menu` of `nldd-popover` wordt automatisch verankerd en getoggeld, inclusief `expanded`/`aria-haspopup`-sync. `FormatToolbar.vue` doet dat nu allemaal met de hand voor het kopjes-menu: `headingTriggerRef`, `headingMenuRef`, `headingOpen`, `menu.anchorElement = trigger` en een eigen toggle-handler. Dat kan grotendeels weg. Ook nalopen waar de trigger een nldd-button is (o.a. `ReactionBar.vue`). Niet van toepassing: `MentionProfilePopover`/`ReactorPopover` (anker is een willekeurig element in de tekst) en `SendButton.vue` (blijft bewust custom vanwege de primary-variant-styling van nldd-split-button, ongewijzigd in 0.8.69). Afhankelijk van PR #124 (bump naar ^0.8.69).

robbertbos added the

nldd ds

label

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

robbertbos commented

2026-08-05 21:35:25 +00:00

Author

Owner

Duplicate of #170, which covers the same sweep app-wide. Closing here so there is one place to track it.

Verified where it actually applies in the composer: FormatToolbar.vue has two nldd-menus wired by hand through a shared openMenu() helper (lines 154-161, call sites 220 and 392), setting anchorElement and calling showPopover(), with aria-haspopup/aria-expanded maintained manually on a native

Duplicate of #170, which covers the same sweep app-wide. Closing here so there is one place to track it. Verified where it actually applies in the composer: FormatToolbar.vue has two nldd-menus wired by hand through a shared openMenu() helper (lines 154-161, call sites 220 and 392), setting anchorElement and calling showPopover(), with aria-haspopup/aria-expanded maintained manually on a native <button>. That is exactly what the popup slot replaces, and it is in scope for the NLDD pass now running under #170.

robbertbos closed this issue

2026-08-05 21:35:25 +00:00