suggestion: replace usage of `router.replace` with `router.push` in various places
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
- 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
- Vorherrschende Sprache
- Rust
- Sterne
- 2.5k
- Forks
- 663
- Ø Merge
- 2 T. 11 Std.
- Gemergte PRs (30 T.)
- 122
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus modrinth/code
-
app
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag
-
website
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
modrinth/code#8256 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
modrinth/code#8253 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Translation errorOffenwebsite
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag
-
Button reorder requestEvtl. wieder frei @Ceebug hat das vor 17 Tagen übernommen, und es ist kein Pull Request offen. Offen
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 70/100
modrinth/code#8136 · 10 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
pop-os/cosmic-settings#2296 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Eyevinn/open-live-ingest#36 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
superradcompany/microsandbox#2071 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug bug jury peripheral:i2s
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
esp-rs/esp-hal#6828 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
Maintainer antworten meist innerhalb von 1 Tag