@fluentui/web-components: fluent-menu-list loses its arrow keys after it is moved in the DOM (Firefox, Safari)
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 84/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript, web-dev
- Ambito
- accessibility, frontend
Direzione di ricerca
La issue descrive il comportamento in menu-list.js: disconnectedCallback() disconnette il gruppo di focus, mentre setItems() lo riutilizza dopo la riconnessione. Leggi questi metodi e il ciclo di vita del gruppo di focus descritto nella issue, poi riproduci lo spostamento del DOM in Firefox o Safari. Il lavoro è completato quando ArrowDown sposta il focus tra gli elementi del menu dopo lo spostamento, come fa senza lo spostamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Version: @fluentui/web-components 3.1.3, @microsoft/focusgroup-polyfill 1.6.0. Engines: Firefox 153 and WebKit 26.6 (Playwright's builds) fail; Chromium 153 passes, because it has native focusgroup and the polyfill stands aside.
Summary
A fluent-menu-list builds its focus group (the polyfill that gives the arrow keys in an engine without native focusgroup) on its first setItems(), disconnects it when the list leaves the document, and never builds another. Any DOM move, which disconnects and reconnects the element (for example insertBefore, or a framework reordering a list), leaves a menu whose arrow keys do nothing: focus stays on the first item, so Enter chooses the first item whatever the user wanted.
Reproduction
<!doctype html>
<div id="host">
<fluent-menu>
<fluent-button slot="trigger">Actions</fluent-button>
<fluent-menu-list>
<fluent-menu-item>One</fluent-menu-item>
<fluent-menu-item>Two</fluent-menu-item>
<fluent-menu-item>Three</fluent-menu-item>
</fluent-menu-list>
</fluent-menu>
</div>
<script type="module">
import '@fluentui/web-components/button.js';
import '@fluentui/web-components/menu.js';
import '@fluentui/web-components/menu-list.js';
import '@fluentui/web-components/menu-item.js';
// Move the menu in place once, as a framework's reorder would.
const menu = document.querySelector('fluent-menu');
customElements.whenDefined('fluent-menu-list').then(() => menu.parentElement.insertBefore(menu, menu.nextSibling));
</script>
- Open the page in Firefox or Safari. Tab to "Actions" and press Enter: the menu opens with focus on "One".
- Press ArrowDown twice.
- Expected: focus on "Three" (as without the move, and as in Chrome). Actual: focus stays on "One".
The standalone page above reproduces it as written (bundled with esbuild, served to each engine): never moved, "One", "Two", "Three" in all three engines; moved in place once, "One", "Two", "Three" in Chromium and "One", "One", "One" in Firefox and WebKit.
Cause
node_modules/@fluentui/web-components/dist/esm/menu-list/menu-list.js:15-18:disconnectedCallback()callsthis.fg?.disconnect(), and keepsthis.fg.menu-list.js:19-34:setItems()creates theFocusGrouponlyif (!this.fg); otherwise it callsthis.fg.update().node_modules/@microsoft/focusgroup-polyfill/build/focusgroup-shadowless.mjs:101-114: the constructor adds thekeydown,focusinandfocusoutlisteners, once;:127-132:disconnect()aborts them and sets the owner tonull;update()returns at once when there is no owner (:146).
So after a reconnection setItems() "updates" a disconnected group, which does nothing.
Suggested fix
Drop the group when disconnecting (this.fg?.disconnect(); this.fg = undefined;), so the next setItems() builds a new one; or build the group in connectedCallback.
- Lingua principale
- TypeScript
- Stelle
- 20.3k
- Fork
- 2.9k
- Merge medio
- 2g 10h
- PR unite (30g)
- 72
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di microsoft/fluentui
-
[Bug]: Button shims in react-migration-v8-v9 always render the ms-Button--primary classForse già presa @jakesalvesen l’ha presa 1 giorno fa. ApertaFluent UI react-components (v9) Needs: Triage :mag: Type: Bug :bug:
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: AreaChart renders SVG rect with negative width and height on initial renderForse già presa @huong8373tt-beep l’ha presa 1 giorno fa. ApertaNeeds: Triage :mag: Package: charting Type: Bug :bug:
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: Vertical Slider still logs the deprecated 'slider-vertical' warning in Chromium 132+Forse già presa @jakesalvesen l’ha presa 4 giorni fa. ApertaFluent UI react-components (v9) Needs: Triage :mag: Type: Bug :bug:
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
microsoft/fluentui#36945 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di microsoft/fluentui
Issue simili
-
area: image-engine bug priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
snapotter-hq/SnapOtter#3885 ·
I maintainer di solito rispondono entro 1 giorno
-
Mend: dependency security vulnerability
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 3 giorni
-
pending triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
bug P3 UI/UX
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
openclaw/openclaw#182077 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno