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

Aberta Para iniciantes
  • Algo errado?
#15,170 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
78/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
javascript, scss
Domínio
accessibility, frontend

Direção de pesquisa

O relatório aponta para src/resources/filters/layout/lightbox.lua e src/resources/formats/html/_quarto-rules.scss; leia primeiro as regras de encapsulamento do lightbox e de ocultação de temas. Reproduza com o exemplo QMD fornecido e, em seguida, verifique se o link do tema desativado está oculto na ordem de tabulação do teclado e na árvore de acessibilidade, enquanto a imagem correspondente continua abrindo. Nenhum arquivo de teste é mencionado na issue.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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
Linguagem predominante
JavaScript
Estrelas
6.1k
Forks
476
Merge médio
17h 7min
PRs com merge (30d)
112

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de quarto-dev/quarto-cli

Todas as issues de quarto-dev/quarto-cli

Issues semelhantes

Mais issues de JavaScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.