Etapa 17 fase2 sweep orange to brand primary#288
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Warning Review limit reached
Your plan currently allows 5 reviews/hour. Refill in 16 minutes and 48 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f7a7d58394
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (ms < 200) return 'text-emerald-400'; | ||
| if (ms < 600) return 'text-amber-400'; | ||
| if (ms < 1500) return 'text-orange-400'; | ||
| if (ms < 1500) return 'text-brand-primary-400'; |
There was a problem hiding this comment.
Replace invalid
text-brand-primary-400 utility
latencyClass now returns text-brand-primary-400, but tailwind.config.ts defines brand-primary only with DEFAULT, hover, active, glow, and foreground keys (no numeric 400 shade). In this state, the class will not resolve to a generated utility, so the 600–1499ms latency branch can render without the intended color styling.
Useful? React with 👍 / 👎.
* Etapa 17 fase2 sweep orange to brand primary (#288) * refactor(theme): sweep -orange* → -brand-primary* (chunk 1/8 — Etapa 17 fase 2) * refactor(theme): sweep -orange* → -brand-primary* (chunk 2/N — Etapa 17 fase 2) * ci: harden deploy gate baselines (#287) Co-authored-by: Codex Simulation <codex-simulation@example.local> --------- Co-authored-by: Codex Simulation <codex-simulation@example.local>
* fix(contracts): remove duplicate CONTRACTS import * fix(auth): preserva isConfig p/ provider desabilitado (provider_is_not_enabled) mapOAuthError passou a retornar mensagem PT-BR amigável, mas isso fazia resolveOAuthError cair no fallback genérico (isConfig=false), reexibindo o botão "Tentar Google Novamente" para uma falha de CONFIG que não tem solução client-side. Adiciona heurística em resolveOAuthError que reconhece "não está habilitado"/"not enabled" → copy provider_is_not_enabled (isConfig). [Codex review] https://claude.ai/code/session_01SN7BG6cYaPS1t1fYLkga4Z * fix(ui): gateia transforms do botão Google atrás de motion-safe Mesmo bug do login-submit (#3): o botão social-login-google tinha 'transition-all hover:scale-[1.01] active:scale-[0.98]' SEM gate motion-safe. Sob o reducedMotion:'reduce' do CI o transform seguia animando a cada hover/mousedown -> 'locator.click: Timeout' no smoke 40 (e o waitForRequest era sintoma do clique nao registrar). Gateado, o botao fica estatico e clicavel sob reduced-motion. * test(contracts): port reusable webhook coverage * chore(ci): refresh lint and typecheck baselines * codex 1 (#290) * Etapa 17 fase2 sweep orange to brand primary (#288) * refactor(theme): sweep -orange* → -brand-primary* (chunk 1/8 — Etapa 17 fase 2) * refactor(theme): sweep -orange* → -brand-primary* (chunk 2/N — Etapa 17 fase 2) * ci: harden deploy gate baselines (#287) Co-authored-by: Codex Simulation <codex-simulation@example.local> --------- Co-authored-by: Codex Simulation <codex-simulation@example.local> --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Codex <codex@local> Co-authored-by: Codex Simulation <codex-simulation@example.local>
📋 Descrição
🎯 Tipo de mudança
🔗 Issues relacionadas
Closes #
Refs #
🌐 Sistemas afetados
🧪 Como testar
✅ Checklist pré-merge
Qualidade
npx tsc --noEmitpassa sem errosnpm run test)Segurança
console.logcom payloads sensíveis (usarlogger.*)Documentação
mem://) se a mudança afetar arquitetura/regras_backup_*_YYYYMMDDse destrutivasUI
📸 Screenshots (se UI)
🔄 Plano de rollback
Summary by cubic
Trocamos o uso de
orangeporbrand-primaryem componentes da UI para padronizar a paleta e alinhar com o design. Mudança visual apenas; sem impacto funcional.text-brand-primary.Shieldcomtext-brand-primary.text-brand-primary.bg-brand-primary.bg/text/borderembrand-primary.text-brand-primary-400.Written for commit f7a7d58. Summary will update on new commits. Review in cubic