a11y: lightbox link of an off-theme `.light-content` or `.dark-content` image stays focusable

Abierto Apto para principiantes
  • ¿Algo no está bien?
#15,170 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
javascript, scss
Área
accessibility, frontend

Línea de trabajo

El informe apunta a src/resources/filters/layout/lightbox.lua y src/resources/formats/html/_quarto-rules.scss; lee primero las reglas de envoltura de lightbox y de ocultación de temas. Reproduce el comportamiento con el ejemplo QMD proporcionado y luego verifica que el enlace del tema desactivado esté oculto en el orden de tabulación del teclado y en el árbol de accesibilidad, mientras la imagen correspondiente sigue abriéndose. En el issue no se menciona ningún archivo de prueba.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

accessibility bug
I have:
  • searched the issue tracker for similar issues
  • installed the latest version of Quarto CLI
  • formatted my issue following the Bug Reports guide
Bug description

With a light and a dark version of an image, Quarto hides the one that does not match the colour scheme, but it leaves the lightbox link around it.
The hidden image's link is zero-size and invisible, yet it stays in the keyboard tab order, and pressing Enter on it opens the lightbox with the image from the other colour scheme.
Keyboard users tab onto an element they cannot see.

The lightbox filter wraps every lightbox image in an <a class="lightbox"> and puts no theme class on the link.

The stylesheet hides the image with display: none !important, and the rule only matches the image.

#14691 describes the same pattern for the dashboard navbar logo.

Steps to reproduce
---
title: "Lightbox light and dark test"
lightbox: auto
format:
  html:
    theme:
      light: default
      dark: darkly
---

![Light version](light.svg){.light-content}

![Dark version](dark.svg){.dark-content}

Render it, press Tab until the focus reaches the second image in light mode, then press Enter.

Actual behavior

The output contains one link per image, and the link of the off-theme image measures 0 by 0:

<a href="dark.svg" class="lightbox" data-gallery="quarto-lightbox-gallery-2" title="Dark version"><img src="dark.svg" class="dark-content img-fluid figure-img" alt="Dark version"></a>

In light mode the tab order is: skip link, light image link, dark image link (invisible), lightbox close button.
In dark mode it is the same with the roles swapped.
Enter on the invisible link opens the lightbox with the hidden image.

Expected behavior

The link of the hidden image is hidden with it, so it leaves the tab order and the accessibility tree.

Your environment
  • IDE: none, terminal only
  • OS: macOS (Darwin 27.0.0, Apple Silicon)
Quarto check output
Quarto 99.9.9
[✓] Checking versions of quarto binary dependencies...
      Pandoc version 3.10.0: OK
      Dart Sass version 1.101.0: OK
      Deno version 2.7.14: OK
      Typst version 0.15.1: OK
[✓] Checking Quarto installation......OK
      Version: 99.9.9
      commit: 69592e5c1718d3e1b2f3d11e757d32e23a16af67
Lenguaje dominante
JavaScript
Estrellas
6.1k
Forks
478
Merge medio
17 h 7 min
PR fusionados (30 d)
112

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de quarto-dev/quarto-cli

Todos los issues de quarto-dev/quarto-cli

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.