Skip to content

feat(portal): add navbar hover underline#265

Merged
danielhe4rt merged 1 commit into
he4rt:4.xfrom
andrefelipe18:feat/navbar-hover-underline
May 20, 2026
Merged

feat(portal): add navbar hover underline#265
danielhe4rt merged 1 commit into
he4rt:4.xfrom
andrefelipe18:feat/navbar-hover-underline

Conversation

@andrefelipe18
Copy link
Copy Markdown
Contributor

@andrefelipe18 andrefelipe18 commented May 20, 2026

2026-05-20.17-37-18.mp4

Description

This pull request introduces a new navbar component for the portal module with an animated underline hover effect. The navbar displays navigation links (Comunidade, Sobre, Projetos, Depoimentos, Contato) and a Discord button. The links feature a smooth CSS animation that expands an underline bar on hover and focus states, using Tailwind CSS utilities and CSS custom properties.

References

Related recent changes:

  • PR #264: Centralize display timezone config

Dependencies & Requirements

No new dependencies or configuration changes required. The feature uses only existing Tailwind CSS utilities and CSS pseudo-elements.

Contributor Summary

Contributor Lines Added Lines Removed Files Changed
andrefelipe18 43 0 1

Changes Summary

File Path Change Description
app-modules/portal/resources/views/components/navbar.blade.php New navbar component with navigation links and animated underline hover effect using after: pseudo-elements with transition animations tied to --primary CSS variable

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 5f84d228-494f-4ae6-ba4e-7025ab0ade5b

📥 Commits

Reviewing files that changed from the base of the PR and between c35082e and a304e52.

📒 Files selected for processing (1)
  • app-modules/portal/resources/views/components/navbar.blade.php

📝 Walkthrough

Walkthrough

The navbar component link styling has been enhanced with an animated underline effect. The anchor element's class attribute was expanded to use Tailwind after:* pseudo-element utilities that create a positioned underline with animated width and background color tied to the --primary CSS variable. The update also includes improved transition, duration, and focus-visible styling to replace the prior simpler hover-only behavior.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a navbar hover underline effect with animated styling and focus-visible behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Member

@gvieira18 gvieira18 left a comment

Choose a reason for hiding this comment

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

LGTM! 👍🏻

Copy link
Copy Markdown

@daisukedd daisukedd left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown

@sirelves sirelves left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@danielhe4rt danielhe4rt left a comment

Choose a reason for hiding this comment

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

LGTM! Herói NACIONAL!!

Copy link
Copy Markdown
Member

@Clintonrocha98 Clintonrocha98 left a comment

Choose a reason for hiding this comment

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

LGTM!!!!!!!!!

amei esse PR!!!!

Copy link
Copy Markdown

@victor-reis-dev victor-reis-dev left a comment

Choose a reason for hiding this comment

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

lgtm

@maysaraujo
Copy link
Copy Markdown

LGTM

@luxu
Copy link
Copy Markdown

luxu commented May 20, 2026

LGTM!

1 similar comment
@cesarbosqueiro
Copy link
Copy Markdown

LGTM!

@danielhe4rt danielhe4rt merged commit efd05ff into he4rt:4.x May 20, 2026
6 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.