Skip to content

refactor(frontend): centralizar tokens de cor em tokens.css (DS-06) - #87

Open
JoyceYasminSilva5 wants to merge 3 commits into
developfrom
feature/ds-06-tokens
Open

JoyceYasminSilva5 wants to merge 3 commits into
developfrom
feature/ds-06-tokens

Conversation

@JoyceYasminSilva5

@JoyceYasminSilva5 JoyceYasminSilva5 commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

O que é

Move as variáveis de cor do painel para um arquivo só (tokens.css). Não muda a aparência. A identidade v4 NÃO foi aplicada aqui (isso é da DS-08).

O que mudou

  • frontend/src/styles/tokens.css (novo): 14 variáveis de cor do tema escuro e 13 do claro, com os mesmos valores de antes.
  • frontend/src/App.css: importa o tokens.css na primeira linha; as duas primeiras regras ficam só com as propriedades que não são variáveis.
  • Não alterei index.css, api.js, hooks, mockApi.js nem backend.
  • Fontes e espaços ficam para a DS-07. O apontamento no DESIGN.md entra na DS-08.

O que foi conferido

  • No PC da autora: npm run lint, npm test (19 passed) e npm run build passaram.
  • Script comparou as regras do CSS antes e depois: 63 seletores, mesmas propriedades e valores.
  • Conferência automatizada feita pela IA (Chromium sem interface, Linux): build em modo mock antes e depois, login como gestor.joyce, 5 áreas, temas claro e escuro, 1440 e 390 px, cenários normal, erro e falha-trilha. Estilos computados idênticos em 78 de 78 estados.
  • Conferência visual da autora no navegador: painel aberto em localhost:5173, tema escuro, botão "Tema claro" e aba Diagnóstico sem mudança visível (informado pela autora).

Não verificado

  • Hover e foco por teclado, perfis suporte/analista, cenários vazio e lento, fonte Segoe UI real, Opera GX.
  • O CSS final do build não é idêntico byte a byte (duas regras viraram quatro, mesmas declarações, mesma ordem).
  • O apontamento no DESIGN.md pedido pelo cartão ficou de fora.
  • Pedido ao revisor: dar uma olhada no navegador.

Refs #31

@Vinny1313

Copy link
Copy Markdown
Collaborator

Conferência do Vinícius no navegador (modo mock, perfil gestor), comparando esta branch com a develop:

  • 5 abas (Saúde, Incidentes, Aprovações, Auditoria, Diagnóstico), temas escuro e claro: cor, fundo, borda, contorno, sombra, fonte e cursor de todos os elementos ficaram idênticos.
  • Foco por teclado em todos os botões, links e campos de cada aba: idêntico, com o anel de foco visível.
  • Hover: só existe a regra button:hover com filter: brightness(1.12), que não usa as cores movidas.

Não cobri: hover real com o mouse, perfis suporte e analista, cenários vazio e lento, Segoe UI real e Opera GX.

Joyce, dois pontos:

@Vinny1313

Copy link
Copy Markdown
Collaborator

Revisão de design (10/11/2026), feita com agente de design do Claude e conferida pelo Vinícius. Só leitura; não substitui a revisão humana.

Veredito: pode integrar.

  • Valores comparados por script contra o App.css original: 14 variáveis no escuro e 13 no claro, 0 diferenças.
  • Na branch: npm ci, lint e build passaram, e npm test deu 20 passed (o PR diz 19; a diferença vem da base).
  • O @import funciona no Vite. Nenhum arquivo proibido foi alterado.

Para anotar, Joyce (não bloqueia):

  • O cartão fala em "cores, fontes e espaços" e o PR moveu só as cores. Diga na descrição que fontes e espaços ficam para a DS-07.
  • O apontamento no DESIGN.md deve entrar na DS-08.
  • O comentário "único lugar para ajustar cores" é forte demais se ainda houver cor fixa fora das variáveis. Troque para "lugar das variáveis de cor".
  • Tire do rascunho e atualize a base com a develop.

Troca 'unico lugar para ajustar cores' por 'lugar das variaveis de cor do painel', como pediu a revisao do Vinicius. So muda um comentario; nenhum valor de cor foi alterado. Refs #31
@JoyceYasminSilva5
JoyceYasminSilva5 marked this pull request as ready for review November 10, 2026 21:10
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.

2 participants