Skip to content

test(frontend): alvo de 44 px tolera arredondamento de subpixel - #44

Merged
MatheusNespolo merged 1 commit into
mainfrom
fix/e2e-alvo-44px-subpixel
Oct 4, 2026
Merged

MatheusNespolo merged 1 commit into
mainfrom
fix/e2e-alvo-44px-subpixel

Conversation

@MatheusNespolo

Copy link
Copy Markdown
Owner

O que muda

O job frontend-tests falhou na main depois do merge do #43 (run 37212091664) no teste mobile.spec.mjs › cenário 6: o botão "Visão geral" mediu 43.99998474121094 px contra o mínimo de 44 px. O CSS pede min-height: 44px; a fração vem do layout em subpixel do navegador, não de uma regressão (o mesmo teste passou no CI do PR #43, e nada do frontend mudou).

  • Nova constante ALTURA_MIN_TOQUE = 44 - 0.01 em test/frontend_smoke/helpers.mjs, com comentário (WCAG 2.5.5, 44 × 44 px, tolerância de 0,01 px de arredondamento).
  • As 5 verificações de alvo de toque (mobile.spec.mjs ×3, navegacao.spec.mjs, botoes.spec.mjs) passam a usá-la.
  • Nenhum código de produto muda.

Como validar

  • node --check nos 4 arquivos alterados.
  • O CI deste PR (job frontend-tests) é a validação; localmente o Playwright estoura o tempo limite de 90 s nesta máquina (lentidão já conhecida), sem falha de asserção.

Checklist

  • Rodei as validações do Guia de Contribuição (scripts/precommit-checks) — não executado
  • Não há senhas, tokens, .env ou secrets.h na mudança
  • Atualizei a documentação afetada e o docs/CHANGELOG.md — não se aplica (só teste)
  • Mensagens de commit seguem o padrão tipo: descrição
  • Informei o que não consegui validar: o teste E2E completo localmente (tempo limite), coberto pelo CI

🤖 Generated with Claude Code

…undingBox

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MatheusNespolo
MatheusNespolo merged commit 91d9e6a into main Oct 4, 2026
7 checks passed
@MatheusNespolo
MatheusNespolo deleted the fix/e2e-alvo-44px-subpixel branch October 4, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant