Skip to content

Polish frontend auth and landing UI#67

Merged
diasgarcia merged 4 commits into
mainfrom
codex/refine-auth-and-frontend-ui
May 17, 2026
Merged

Polish frontend auth and landing UI#67
diasgarcia merged 4 commits into
mainfrom
codex/refine-auth-and-frontend-ui

Conversation

@bryancalvarenga

Copy link
Copy Markdown
Contributor

Summary

  • Align login and register pages with the landing page visual language
  • Restore/refine landing, dashboard, and public menu sizing after main changed
  • Soften auth carousel copy, chips, and layout while removing unsupported restaurant-count claims

Tests

  • npm.cmd exec eslint -- 'src/app/(auth)/login/page.tsx' 'src/app/(auth)/register/page.tsx' 'src/features/auth/components/AuthCarousel.tsx' 'src/app/page.tsx' 'src/app/(dashboard)/dashboard/page.tsx' 'src/app/[slug]/page.tsx'
  • npm.cmd test

@diasgarcia

Copy link
Copy Markdown
Collaborator

Resolvido direto nesta PR.

Mantive a tela nova da PR 67, mas removi os data-testid que estavam em elementos invisiveis/fora da jornada atual do usuario. Tambem ajustei os testes para cobrirem apenas o contrato visivel de hoje e deixei TODO ligado a #57 para reativar a cobertura quando esses cards/atalho voltarem como UI real.

Motivo: se o elemento nao esta visivel/utilizavel, o teste nao deve passar so porque encontrou um data-testid escondido. Quando a funcionalidade estiver pronta, a gente reativa o elemento visivel e o teste junto.

@diasgarcia diasgarcia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aprovado.

A tela nova foi preservada e os ajustes finais removeram os data-testid que estavam em elementos invisíveis/fora da jornada atual do usuário. Os testes agora cobrem o contrato visível de hoje, e a pendência ficou documentada para reativar a cobertura quando esses cards/atalho voltarem como UI real.

Validações locais informadas:

  • cd web && npm run build
  • cd web && npm test
  • node --check nos arquivos Playwright alterados

@diasgarcia diasgarcia merged commit 74c583e into main May 17, 2026
3 checks passed
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