Skip to content

Close menu once a left-menu item is clicked - #4614

Merged
ildyria merged 1 commit into
LycheeOrg:masterfrom
jphuguet:feat/auto-close-left-menu
Aug 14, 2026
Merged

Close menu once a left-menu item is clicked#4614
ildyria merged 1 commit into
LycheeOrg:masterfrom
jphuguet:feat/auto-close-left-menu

Conversation

@jphuguet

@jphuguet jphuguet commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Close the left menu once an item has been clicked, for instance Admin

Summary by CodeRabbit

  • Bug Fixes
    • The left navigation menu now closes automatically after navigating to a different page.
    • Improved menu behavior across both supported interface versions.

@jphuguet
jphuguet requested a review from a team as a code owner August 14, 2026 15:22
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d3d9c3d5-ff8a-4353-9726-c67fe35f5319

📥 Commits

Reviewing files that changed from the base of the PR and between 142c330 and a7cf147.

📒 Files selected for processing (2)
  • resources/js/v7/menus/LeftMenu.vue
  • resources/js/v8/menus/LeftMenu.vue

📝 Walkthrough

Walkthrough

Both V7 and V8 left menus now watch route.fullPath and close when navigation changes the route.

Changes

Menu navigation behavior

Layer / File(s) Summary
Route-driven menu closure
resources/js/v7/menus/LeftMenu.vue, resources/js/v8/menus/LeftMenu.vue
Each left-menu implementation watches route.fullPath and sets left_menu_open to false after navigation.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to a7cf1

This is a localized change to close the left menu after selecting an item, with no actionable merge-blocking risk remaining beyond normal checks and review.

Poem

I’m a rabbit with menus to close,
When the route hops, the slideover goes.
V7 and V8 tuck away,
Ready for the next bright day. 🐇

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@ildyria ildyria changed the title close menu once a left-menu item is clicked Close menu once a left-menu item is clicked Aug 14, 2026
@ildyria
ildyria merged commit ed5ad3b into LycheeOrg:master Aug 14, 2026
22 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.

3 participants