`A` active does not account for search params
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
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
Línea de trabajo
Empieza reproduciendo el problema en el ejemplo de StackBlitz enlazado con varios enlaces que solo se diferencien por los parámetros de búsqueda. Inspecciona el comportamiento del router con active-link y aria-current; se considera terminado cuando los enlaces con distintos valores de consulta no aparecen todos marcados como active o aria-current="page".
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
A list of links that are used for filtering content will result in all of them having an active class and all of them having aria-current="page"
<A href="/?example=1">1</A>
<A href="/?example=2">2</A>
<A href="/?example=3">3</A>
The problem is that all 3 links cannot be active here at the same time and aria-current fails to differentiate this difference. The workaround here would be to build a custom A that also takes into account the search params.
Your Example Website or App
https://stackblitz.com/edit/solidjs-templates-hhpgedwq?file=src%2FApp.tsx
Steps to Reproduce the Bug or Issue
- Create multiple links that update search params
- See all are considered active and all have an invalid
aria-current
Expected behavior
Some way to force active to account for the search params
Screenshots or Videos
No response
Platform
- OS: macOS
- Browser: Chrome
- Version: 132
Additional context
No response
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.3k
- Forks
- 180
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 27
Preparar el entorno
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 solidjs/solid-router
-
Dificultad 4/5 3-5 días Aptitud para principiantes 50/100
solidjs/solid-router#625 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Query enumeration recomputes on pathname/hash-only navigation with unchanged search (Solid 2)Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
solidjs/solid-router#624 ·
Los mantenedores suelen responder en 1 día
-
<A> costs ~6us of server CPU per instance during SSR (20x a plain <a>), mostly mergeProps/splitPropsAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
solidjs/solid-router#583 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
solidjs/solid-router#569 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
solidjs/solid-router#502 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de solidjs/solid-router
Issues similares
-
triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
mermaid-js/mermaid-live-editor#2053 ·
Los mantenedores suelen responder en 1 día
-
factory
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
jessepollak/home#1455 ·
Los mantenedores suelen responder en 1 día
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
lingdojo/kana-dojo#31227 · 1 comentario · 5 reacciones ·
Los mantenedores suelen responder en 1 día
-
mobile: device viewer shows dark status bar icons on its dark backdrop in light mode (Android)Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
appandflow/stim#1838 ·
Los mantenedores suelen responder en 1 día