-
Notifications
You must be signed in to change notification settings - Fork 1
Melhorar visual e organização do conteúdo #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Melhorar visual e organização do conteúdo #6
Conversation
Implementa proposta de melhorias visuais mantendo 100% do texto original: Componentes criados: - MetricPill: pills de métricas no Hero - MetricCard: cards grandes de métricas - StyleBar: barras de progresso para estilos - ContribCard: cards de contribuição detalhados Mudanças na home: - Hero: adiciona 3 métricas visuais (cantorias/cantadores/estilos) - O Que É Repente: adiciona vídeo embed do YouTube - Nova seção: Acervo em Números com dashboard visual - Contribua: reorganiza em 3 cards detalhados Cálculos dinâmicos: - Total de cantadores via agregarCantadores() - Top 5 estilos mais frequentes com contagem - Barras proporcionais no gráfico Arquivos criados: - view/src/components/hero/MetricPill.tsx - view/src/components/dashboard/MetricCard.tsx - view/src/components/dashboard/StyleBar.tsx - view/src/components/contribute/ContribCard.tsx - MELHORIAS-VISUAIS-IMPLEMENTADAS.md - IMPLEMENTACAO-COMPLETA.md Arquivo modificado: - view/src/routes/home.tsx (6 edições) Garantia: Todo o texto original foi preservado.
|
Cursor Agent can help with this pull request. Just |
Nova seção destacando a relação entre repente e música nordestina:
Conteúdo:
- Título: "O Repente é o DNA Original da Música Nordestina"
- Explica que repente é a raiz da música nordestina
- Menciona artistas influenciados:
* Luiz Gonzaga ("O Rei do Baião")
* Alceu Valença ("Martelo Alagoano")
* Zé Ramalho (métricas e temas)
- Destaca "relação amistosa e respeitosa"
- Explica "linhas que se confundem"
- Callout sobre como Vilanova explora essa influência
Layout:
- Grid 60/40 (texto + card de artistas)
- 2 CTAs: link para /musicas e /cantorias
- Responsivo (empilha no mobile)
Posicionamento:
- Entre "Acervo em Números" e "Os Cantadores"
Arquivos:
- view/src/routes/home.tsx (1 seção adicionada)
- NOVA-SECAO-MUSICA-POPULAR.md (documentação)
|
@cursoragent quero que:
|
|
Perfeito! Vou criar um resumo visual para você: ✅ Melhorias no Hero Implementadas!🎯 3 Melhorias Concluídas1. ✅ Métricas Agora São Links ClicáveisAntes: Pills estáticas
2. ✅ MUITO Mais Destaque VisualMudanças dramáticas:
Hover Effects Adicionados:
Resultado: Métricas agora DOMINAM o Hero! 3. ✅ Textos Menores, Informações Importantes em DestaqueReduções de tamanho:
Textos condensados (~20% de redução): Antes:
Depois:
Destaques visuais:
📊 Resultado Visual🎯 Benefícios✅ Navegação facilitada - clica e vai direto 🚀 Testar AgoraAbra: http://localhost:8787
📝 CommitArquivos modificados:
Tudo pronto! As métricas agora são DESTAQUE total e clicáveis! 🎉🚀 Learn more about Cursor Agents |
Implementa 3 melhorias solicitadas no PR #6: 1. Métricas agora são links clicáveis: - 15 Cantorias → /cantorias - 12 Cantadores → /cantadores - 6 Estilos → /estilos - Usa Link do TanStack Router 2. Muito mais destaque visual: - Números: text-2xl → text-4xl md:text-5xl - Padding: px-4 py-2 → px-6 py-4 - Borda: border-2 marrom → border-3 vermelha - Shadow: shadow-sm → shadow-lg - Labels: text-sm → text-base md:text-lg font-semibold - Hover: scale-105 + shadow-xl + border color change - Gap: 2 → 3 3. Textos menores com informações importantes em destaque: - Subtítulo: text-xl md:text-2xl → text-lg md:text-xl - Parágrafos: text-xl md:text-2xl → text-lg md:text-xl - Redução de ~20% no texto (mesma mensagem) - "Somos open source" agora em vermelho (text-[#C84B31]) - Keywords mantidas em bold para destaque - space-y-6 → space-y-4 (mais compacto) Arquivos modificados: - view/src/components/hero/MetricPill.tsx - view/src/routes/home.tsx - HERO-METRICAS-DESTAQUE.md (documentação) Resultado: Métricas 2-3x maiores, clicáveis, com hover effects, e textos mais concisos focando no essencial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| to="/cantorias" | ||
| className="inline-flex items-center justify-center gap-2 px-6 py-3 border-2 border-[#4A7C59] text-[#4A7C59] font-semibold rounded-lg hover:bg-[#4A7C59]/10 transition-all duration-300" | ||
| > | ||
| Ver Cantorias Originais |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Dead Links Plague Music Section
Links to /musicas route that doesn't appear to exist in the codebase. Multiple links point to /musicas and /musicas/martelo-alagoano-alceu-valenca, but no corresponding route definition exists. Users clicking these links will get 404 errors or routing failures, breaking the user experience for a prominently featured section.
🚀 Preview Deployment Ready!Your changes have been deployed to a preview environment: This preview will be automatically updated with new commits to this PR. Deployed from commit: |
| <div className="flex flex-wrap justify-center md:justify-start gap-4 mb-8"> | ||
| <MetricPill value={15} label="Cantorias" to="/cantorias" /> | ||
| <MetricPill value={totalCantadores} label="Cantadores" to="/cantadores" /> | ||
| <MetricPill value={6} label="Estilos" to="/estilos" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Estilos count: Hard-coded value wrong.
The hard-coded value 6 for "Estilos" is incorrect. The estilos.json file contains only 5 estilos (Galope à Beira-Mar, Oitava, Martelo Alagoano, Desafio, and Décima), and the metadata confirms "total_estilos": 5. The value should be dynamically computed from estilosData.estilos.length to stay in sync with the actual data, or hard-coded as 5 if dynamic loading isn't desired.
Fazendo a proposta de #5
Implement visual and hierarchy improvements to the home page to enhance scannability and impact while preserving 100% of the original content.
Note
Revamps the home page with metric pills, a dashboard, a new music-influence section, and reorganized contribution cards; adds CI permissions and output parsing to post preview URLs on PRs; bumps Deco packages.
MetricPill.MetricCardandStyleBar(top styles computed fromacervoData)./musicasand/cantorias.ContribCard; adjusts typography and copy emphasis.components/hero/MetricPill.tsx,components/dashboard/MetricCard.tsx,components/dashboard/StyleBar.tsx,components/contribute/ContribCard.tsx..github/workflows/deploy-preview.ymlto addpermissions, capturedeco deployoutput, extract preview URL, and expose it aspreview_urlfor PR comments.@deco/workers-runtimeanddeco-cliversions inpackage.json.Written by Cursor Bugbot for commit aae00fd. This will update automatically on new commits. Configure here.