Mobile bottom-nav highlight never syncs with the current route
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 83/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Lee ui/src/components/Navbar.tsx, especialmente las líneas 19–34 y la navegación de escritorio en la línea 87. Sigue cómo pathname determina la pestaña móvil activa en /, en las subrutas de datos y al navegar con el botón de volver o mediante un enlace profundo. El trabajo estará terminado cuando el resaltado móvil se mantenga sincronizado con la ruta actual en esos casos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
In ui/src/components/Navbar.tsx:19-21 mobileNavValue is initialized once from PAGES.findIndex(p => p.route === pathname) and only updated inside handleMobileNavChange (lines 28-34). There is no effect syncing it to pathname. On the index route '/' (and on data subroutes like '/data/sleep') findIndex returns -1, so no tab is highlighted, and any navigation that doesn't go through the bottom nav (initial load, back button, deep link) leaves the highlighted tab wrong. The desktop nav (line 87) reads pathname directly and is fine; the mobile nav should do the same or add a useEffect on pathname.
Filed from a repo-wide code audit (working tree at bcad746).
- Lenguaje dominante
- Rust
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 GoogleBot42/podd
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
GoogleBot42/podd#63 ·
-
documentation
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
GoogleBot42/podd#60 ·
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
GoogleBot42/podd#61 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
GoogleBot42/podd#58 ·
Todos los issues de GoogleBot42/podd
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
rubys/roundhouse#1917 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
pop-os/cosmic-applets#1619 ·
Los mantenedores suelen responder en 3 días
-
binding:python bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día