Drawer fork lets parent contexts leak in (no LayerContentBoundary)
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- react, typescript
- Ambito
- frontend
Direzione di ricerca
Start with src/forks/Drawer/Drawer.tsx and compare its context behavior with Astryx 0.7.2's core Drawer, which wraps content in LayerContentBoundary. The issue gives two possible directions: request an exported boundary or move Modal and the notification stack onto the top layer so the fork can be removed. First determine which path maintainers choose; done means parent contexts no longer leak into drawer content, with the upstream test from #88 enabled or an equivalent check.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Since Astryx 0.7.2, core's Drawer wraps its content in a private LayerContentBoundary, so contexts such as a disabled ButtonGroup or SizeContext around the trigger do not leak into the drawer. ui-common's Drawer fork (src/forks/Drawer/Drawer.tsx) cannot import that boundary, so those contexts still reach the drawer's content, as they did with lab's Drawer. The upstream test for it is skipped in #88.
Options:
- ask Astryx to export
LayerContentBoundary(or an equivalent reset), then use it in the fork; - move ui-common's
Modaland the notification stack onto the top layer so the fork's reasons go away and it can be deleted in favour of core's Drawer.
- Lingua principale
- TypeScript
- Stelle
- 2
- Fork
- 0
- Merge medio
- 1g 1h
- PR unite (30g)
- 35
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la 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 lablup/ui-common
-
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 66/100
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à 4/5 3-5 giorni Idoneità per principianti 43/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
lablup/ui-common#134 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di lablup/ui-common
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
hugomrdias/playwright-test#753 ·
I maintainer di solito rispondono entro 1 giorno
-
Review view: Daily/Weekly Review header buttons use the app-wide toolbar control styleForse già presa @colherdesopa l’ha presa oggi. Aperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
dongdongbh/Mindwtr#1515 ·
I maintainer di solito rispondono entro 1 giorno
-
pending-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
kirodotdev/Kiro#12301 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100