Consolidate “dialog requested” events
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- html, javascript
- Ambito
- frontend
Direzione di ricerca
Read app/static/js/events.js and the dialog-triggering section of app/static/js/app.js first, then inspect the menu-bar references and the wifi-dialog example. Consolidate the dialog request events around the shared event and update external menu-bar injections; done means dialog requests use the common event path without the individual event ids or menu-bar dependency.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We currently have a significant number of events that trigger opening a dialog – e.g. shutdown-dialog-requested, update-dialog-requested, etc. We also currently register these events at <menu-bar> level, not at document level.
The downsides of this are that it’s hard to keep track of all these individual event ids (which we must ensure are globally unique), and that we cannot trivially trigger a dialog from anywhere outside the <menu-bar> (e.g., from another dialog). See discussion at https://github.com/tiny-pilot/tinypilot/pull/1860.
So we could consolidate all those ...-dialog-requested events into a single DialogRequestedEvent that would live in events.js. That way, we reduce the overall number of event ids that we have to manage, and we can simplify our event handler registration code.
When we do this, we should also refactor any places outside of app.js where we reference the <menu-bar> to inject events directly – such as in the <wifi-dialog>.
- Lingua principale
- Python
- Stelle
- 3.5k
- Fork
- 292
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
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 tiny-pilot/tinypilot
-
bug medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
tiny-pilot/tinypilot#1419 ·
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
tiny-pilot/tinypilot#1929 · 3 commenti ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
tiny-pilot/tinypilot#1899 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
tiny-pilot/tinypilot#1896 ·
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
tiny-pilot/tinypilot#1882 ·
Tutte le issue di tiny-pilot/tinypilot
Issue simili
-
ACK_WAITING HELP_WANTED UPDATE_CS
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OWASP/CheatSheetSeries#2458 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
BasedHardware/omi#19711 ·
I maintainer di solito rispondono entro 1 giorno
-
Qwen3_5MoeModel no longer returns router_logits, breaking aux loss with output_router_logits=TrueAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
huggingface/transformers#49172 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
vllm-project/vllm-metal#885 ·
I maintainer di solito rispondono entro 1 giorno