Overhaul the keyword editor modal lifecycle
I maintainer di solito rispondono entro 1 giorno
@amotl ci sta già lavorando.
Dal 6/11/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Summary
Overhaul the keyword editor modal for the Bootstrap 4 UI. Review how KeywordEditorView and ModalRegion manage the modal and its backdrop. Use a single, consistent modal lifecycle that does not leave the shared modal area open after the editor closes.
Context
The PR review identified overlapping modal ownership and a possible double-backdrop problem. A proposed patch did not resolve the problem: the requester reported that the modal did not become visible and appeared to get stuck while presenting the backdrop. Investigate the cause rather than relying on that patch.
Affected areas include patzilla-ui/navigator/components/keywords/keywords.js, the keyword editor template, and the shared modal region in patzilla-ui/lib/marionette.js.
Acceptance criteria
- The keyword editor opens visibly and can be reopened after it closes.
- Save persists keyword changes, closes the editor, and reapplies highlighting.
- Close and Escape dismiss the editor without leaving an extra backdrop, an open
#modal-area, orbody.modal-openbehind. - Other dialogs that share
#modal-areastill open and close correctly before and after the keyword editor. - Verify these flows in a browser, including an immediate Save after opening.
Requested by @amotl.
PR: https://github.com/ip-tools/patzilla/pull/187
Review thread: https://github.com/ip-tools/patzilla/pull/187#discussion_r4488811889
- Lingua principale
- Python
- Stelle
- 123
- Fork
- 31
- Merge medio
- 4h 1m
- PR unite (30g)
- 74
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 ip-tools/patzilla
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
ip-tools/patzilla#139 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
UI: Update BabelApertadependencies help wanted javascript
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
Show invalid patent numbers removed by normalizationForse già presa @amotl l’ha presa 1 giorno fa. Aperta
ip-tools/patzilla#191 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Re-enable analytics and restore the results modalForse già presa @amotl l’ha presa 2 giorni fa. Aperta
ip-tools/patzilla#190 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di ip-tools/patzilla
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
adafruit/Fruit-Jam-OS#98 ·
-
python triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
microsoft/semantic-kernel#14753 ·
I maintainer di solito rispondono entro 5 giorni
-
area:cli area:configuration documentation priority:low size:small
Difficoltà 1/5 1-3 ore Idoneità per principianti 80/100
RonaldHensbergen/composable-data-stack#1222 ·
I maintainer di solito rispondono entro 1 giorno
-
bug python
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pvlib/pvlib-python#2939 ·
I maintainer di solito rispondono entro 1 giorno