suggestion: replace usage of `router.replace` with `router.push` in various places
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
- 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
- Lenguaje dominante
- Rust
- Estrellas
- 2.5k
- Forks
- 663
- Merge medio
- 2 d 11 h
- PR fusionados (30 d)
- 122
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de modrinth/code
-
app
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
website
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
modrinth/code#8256 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
modrinth/code#8253 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Translation errorAbiertowebsite
Dificultad 1/5 Menos de una hora Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
Button reorder requestQuizá libre de nuevo @Ceebug la tomó hace 17 días y no hay ningún pull request abierto. Abierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 70/100
modrinth/code#8136 · 10 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de modrinth/code
Issues similares
-
bug bug jury peripheral:i2s
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
esp-rs/esp-hal#6828 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
ライブラリを追加Abiertopriority middle
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
KATO-Hiro/AtCoderClans#13015 ·
Los mantenedores suelen responder en 1 día
-
needs-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
UCS-Care-Group/mailcraft#1429 ·
Los mantenedores suelen responder en 1 día
-
priority: medium ready
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
TusanHomichi/PSIV#646 ·
Los mantenedores suelen responder en 1 día