feat(portal): redesign visual do navbar - #529
Conversation
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013mfDhHAzwpnBk94zBvtYV9
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughUpdated desktop navbar color classes to use design tokens. Navigation links now use token-based text and border colors. The store CTA and Discord button use token-based text colors. The “Área do Usuário” button uses Merge Risk: ⚪ Minimal · up to This PR makes a localized visual update to the navbar’s navigation links and CTAs without evidence of functional impact. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description covers the context, main changes, visual evidence, and validation steps. It omits the optional related-issues section and repository commands from the template, but it remains sufficiently complete. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@1pride da uma olhada, agora passa em todos os testes |
|
LGTM! |
|
LGTM! |
|
Lgtm! |
|
Lgtm |
|
LGTM! |
Contexto
O navbar anterior escondia CTAs importantes atrás de detalhes visuais pouco perceptíveis: o botão "Loja" era somente ícone (sem rótulo de texto) e o botão "Área do Usuário" usava estilo outline discreto, sem destaque como ação principal. Esta alteração moderniza o visual do navbar e reforça a hierarquia das CTAs.
Alterações
app-modules/portal/resources/views/components/navbar.blade.phpiconOnly) e passou a ficar agrupado junto aos links de navegação.border-b) com leve elevação no hover.variant="outline"para CTA primário sólido (fundo roxo, texto branco).shadow-md,dark:rounded-xl,dark:shadow-white/5).Plano de Testes
composer run deve validar visualmente o navbar emhttp://localhost:8000(desktop)Evidências
Antes
Depois
Antes
Depois