suggestion: replace usage of `router.replace` with `router.push` in various places
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
Direzione di ricerca
Start with packages/ui/src/layouts/shared/browse-tab/composables/use-browse-search.ts, especially the router.replace call around line 405. The issue describes browser back and forward navigation problems and names Discover tab changes as another example, but the other affected locations are not listed. Find where replacement is used for navigation state and check that switching tabs and search preserve the expected history; done means browser navigation no longer loses the prior tab or search state.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
- I checked the existing issues for duplicate problems
the modrinth website seems to prefer replacing the history state over pushing a new one, which is not standard and breaks browser forward/back navigation, which a portion users would describe as an inconvenience.
one such example of broken functionality is described in https://github.com/modrinth/code/issues/1843, but simpler stuff such as going between tabs in "Discover" (Mods, Shaders, etc.) can overwrite the history state, resulting in losing your place in the previous tab
i will be submitting a pull request to address this issue, starting with the browse search component: https://github.com/modrinth/code/blob/f73010ae0809813120a16b733a3d0bfd23f76e24/packages/ui/src/layouts/shared/browse-tab/composables/use-browse-search.ts#L405
- Lingua principale
- Rust
- Stelle
- 2.5k
- Fork
- 663
- Merge medio
- 2g 11h
- PR unite (30g)
- 122
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 modrinth/code
-
app
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
website
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
modrinth/code#8256 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
modrinth/code#8253 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Translation errorApertawebsite
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
Button reorder requestForse di nuovo libera @Ceebug l’ha presa 17 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 70/100
modrinth/code#8136 · 10 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di modrinth/code
Issue simili
-
bug bug jury peripheral:i2s
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
esp-rs/esp-hal#6828 · 1 commento ·
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
-
ライブラリを追加Apertapriority middle
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
KATO-Hiro/AtCoderClans#13015 ·
I maintainer di solito rispondono entro 1 giorno
-
needs-review
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
UCS-Care-Group/mailcraft#1429 ·
I maintainer di solito rispondono entro 1 giorno
-
priority: medium ready
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
TusanHomichi/PSIV#646 ·
I maintainer di solito rispondono entro 1 giorno