feat(landing-page): add social links and update sections#156
Conversation
WalkthroughUpdated four Blade component views to add navigation links on buttons and social icons, adjust project card links and texts, and replace several heroicons and card titles across contact, hero, projects, and purpose sections. Changes
Sequence Diagram(s)(Skipped — changes are UI/link updates and do not introduce multi-component control-flow requiring a sequence diagram.) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15–20 minutes
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
app-modules/portal/resources/views/components/sections/contact.blade.php(1 hunks)app-modules/portal/resources/views/components/sections/hero.blade.php(1 hunks)app-modules/portal/resources/views/components/sections/projects.blade.php(2 hunks)app-modules/portal/resources/views/components/sections/purpose.blade.php(5 hunks)
🔇 Additional comments (8)
app-modules/portal/resources/views/components/sections/hero.blade.php (2)
37-39: LGTM!The anchor link correctly references the projects section. The outline variant provides appropriate visual distinction from the primary CTA button.
33-35: Discord invite link verified and implementation is correct.The button properly links to the He4rt Developers Discord server with valid syntax and attributes.
app-modules/portal/resources/views/components/sections/purpose.blade.php (1)
10-10: LGTM! Semantic icon and title updates.The updated icons and card titles better represent the community aspects. The changes provide clearer distinction between community features (Comunidade, Interação, Aprendizado, Colaboração).
Also applies to: 23-26, 39-39, 49-52
app-modules/portal/resources/views/components/sections/contact.blade.php (3)
38-40: LGTM!The Discord button correctly links to the community invite with clear, actionable text.
48-50: LGTM!The updated description clearly communicates the purpose of the social media links.
59-82: LGTM! Social media links are properly implemented.All four social links are correctly structured with proper anchor wrapping, consistent styling, and appropriate brand icons. The hover animations provide good UX feedback.
app-modules/portal/resources/views/components/sections/projects.blade.php (2)
30-44: LGTM! Tag update accurately reflects the project nature.The tag change to "Projetos" correctly categorizes the repository's purpose of creating challenges for the community to solve and expand their portfolios. The GitHub repository is active and regularly updated with new challenges.
19-28: LGTM! The button correctly links to the 4noobs project, a repository listing 4noobs content from all categories showing projects developed to facilitate beginner developer learning from the community. The block attribute ensures consistent button sizing across cards, and the "Documentação" tag accurately represents the project's purpose.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
app-modules/portal/resources/views/components/sections/contact.blade.php (1)
52-81: Consider addingtarget="_blank"andrel="noopener noreferrer"for external social links.These external links would benefit from opening in new tabs to keep users on your site. Adding
rel="noopener noreferrer"is a security best practice that prevents the opened page from accessing thewindow.openerproperty.🔎 Example fix for one link (apply to all):
- <a href="https://discord.com/invite/he4rt"> + <a href="https://discord.com/invite/he4rt" target="_blank" rel="noopener noreferrer"> <x-filament::icon icon="fab-discord" class="h-6 w-6 transition-all duration-500 hover:scale-105" /> </a>Additionally, for accessibility, consider adding
aria-labelattributes to describe the link purpose since the icons have no visible text (e.g.,aria-label="Discord").app-modules/portal/resources/views/components/sections/purpose.blade.php (1)
82-88: Good update - grammar fix applied. Consider adding external link attributes.The Portuguese grammar is now correct ("Participe da nossa comunidade"). For consistency with the social links in
contact.blade.php, consider addingtarget="_blank"andrel="noopener noreferrer"if the button component supports these attributes, so users aren't navigated away from the landing page.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
app-modules/portal/resources/views/components/sections/contact.blade.php(1 hunks)app-modules/portal/resources/views/components/sections/purpose.blade.php(5 hunks)
🔇 Additional comments (6)
app-modules/portal/resources/views/components/sections/contact.blade.php (2)
38-40: LGTM!The Discord button now correctly includes the invite link. The implementation is clean and consistent with the component's expected usage.
48-50: LGTM!The updated description provides clear, actionable text for the social links section.
app-modules/portal/resources/views/components/sections/purpose.blade.php (4)
10-10: LGTM!The
heroicon-o-hashtagicon is a good semantic choice for the "Comunidade" (Community) card.
23-26: LGTM!The
heroicon-o-chat-bubble-left-righticon appropriately represents "Interação" (Interaction), and the title rename improves clarity.
39-39: LGTM!The title "Aprendizado" (Learning) pairs well with the code bracket icon and description about open source projects.
49-52: LGTM!The
heroicon-o-user-groupicon and "Colaboração" (Collaboration) title create a cohesive message with the personalized guidance description.
3 participants
This PR adds the following improvements to the landing page
Desafios de Códigoto100DiasDeCodigoon projects sectionSummary by CodeRabbit
New Features
Content Updates
✏️ Tip: You can customize this high-level summary in your review settings.