fix(nav): use pointer cursor on sidebar hamburger toggles

[TheTahsinShahriar]

Summary by CodeRabbit

• Style
  • Improved menu toggle button feedback with pointer cursors, darker hover text, and clearer active-state backgrounds.
  • Standardized toggle interactions across site and admin navigation panels.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 23ab07be-0836-4de0-aeeb-71a7e9133da0

📥 Commits

Reviewing files that changed from the base of the PR and between 96f9d7a and d3866db.

📒 Files selected for processing (1)

• resources/js/components/navigation/Navigation.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

Navigation toggle buttons now use consistent pointer cursors, darker hover text, and active-state backgrounds. SiteRail and AdminRail no longer apply collapse-direction resize cursors.

Changes

Navigation toggle styling

Layer / File(s)	Summary
Toggle interaction styles resources/js/components/navigation/Navigation.tsx	SiteRail, SiteDrawer, and AdminRail toggles now use pointer cursors, darker hover text, and active backgrounds. SiteRail and AdminRail no longer use collapsed or expanded resize cursor classes.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to d3866

Navigation toggles now present consistent pointer, hover, and active styling without changing toggle behavior or accessibility labels. No current merge-readiness risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	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…
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.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: using pointer cursors on sidebar hamburger toggles.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/sidebar-hamburger-cursor

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}