"Try again" on a report run that failed to load is a span, not a button
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #181 de @CRYPTONIKAV — fusionado
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 91/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, react
- Área
- accessibility, frontend
Línea de trabajo
En website/src/ReportsView.jsx, actualiza el elemento “Try again” según lo especificado en el issue. Lee los patrones de aserción existentes en website/test/clickableContent.test.mjs y añade una aserción para la semántica de botón y la posibilidad de enfocarlo. Ejecuta la prueba pertinente; el trabajo estará terminado cuando pase y el control conserve el estilo visual descrito.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When a report run's details fail to load, Reports shows "This run's details could not be loaded. Try again". That "Try again" is a <Box component="span"> with an onClick, so you cannot reach it with the keyboard. Every other "Try again" in website/src is already a real button. This is the only span.
Where: website/src/ReportsView.jsx:
<Box component="span" onClick={() => setMissed(null)} sx={{ ... cursor: "pointer", textDecoration: "underline" }}>Try again</Box>
Done when it is component="button" type="button" with border: 0, bgcolor: "transparent", p: 0, font: "inherit" added to sx, so it looks the same but takes focus.
Test: an assertion in website/test/clickableContent.test.mjs.
Getting started: pip install -e ".[dev]", then python -m pytest -q (offline). UI work: cd website && npm ci && npm test. See CONTRIBUTING.md. One small PR with the change (and its test, where there is one) is perfect.
- Lenguaje dominante
- Python
- Estrellas
- 116
- Forks
- 22
- Merge medio
- 4 h 23 min
- PR fusionados (30 d)
- 19
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 ldbumble/taskuary
-
Dificultad 4/5 3-5 días Aptitud para principiantes 22/100
Los mantenedores suelen responder en 1 día
-
Report ran without Outlook mailAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
ldbumble/taskuary#270 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
enhancement help wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Los mantenedores suelen responder en 1 día
-
bug connector
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
Los mantenedores suelen responder en 1 día
Todos los issues de ldbumble/taskuary
Issues similares
-
URLValidator ignores an explicit empty scheme allowlistPosiblemente ocupada @IlyaKhalafi la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
tortoise/tortoise-orm#2393 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
frappe/frappe_docker#2046 ·
Los mantenedores suelen responder en 1 día
-
Weekly link check failedAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
-
AXI SMU SPEC
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
tenstorrent/tt-oca-harness#3556 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
microsoft/agent-framework#9946 ·
Los mantenedores suelen responder en 1 día