Use semantic HTML for toggle checkboxes
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- playwright
- Área
- accessibility, testing
Línea de trabajo
Start with the linked e2e/security.spec.js lines 11-18 and use its toggle selector to locate the corresponding toggle-container markup in the repository. Update the checkbox and label structure for semantic access, then verify the Playwright security test can target the checkbox normally and that the toggle still works.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We use a bit of an unusual style our toggle-style HTML checkboxes:
<div class="toggle-container">
Hide Sensitive Data
<label class="toggle">
<input type="checkbox" id="include-sensitive-data" />
<span class="toggle-slider"></span>
</label>
</div>
Ideally, it would look a little closer to something like:
<label for="include-sensitive-data">Hide Sensitive Data</label>
<input type="checkbox" class="toggle-slider" id="include-sensitive-data" />
In Playwright, we can't select the checkbox through normal methods because the checkbox itself is technically not visible on the screen, so we have to select the <label> element instead. This likely affects users with screen readers as well.
- Lenguaje dominante
- Python
- Estrellas
- 3.5k
- Forks
- 291
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
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 tiny-pilot/tinypilot
-
bug medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
tiny-pilot/tinypilot#1419 ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
tiny-pilot/tinypilot#1929 · 3 comentarios ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
tiny-pilot/tinypilot#1899 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
tiny-pilot/tinypilot#1896 ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
tiny-pilot/tinypilot#1882 ·
Todos los issues de tiny-pilot/tinypilot
Issues similares
-
bug priority:low
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
CyberAgent/psd2svg#436 ·
-
area/install-update comp/cli comp/desktop P3 sweeper:risk-compatibility type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
NousResearch/hermes-agent#122386 · 1 comentario ·
-
ai-generated
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
vllm-project/production-stack#1105 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100