Skip to content

Fix sidebar navigator initial sync#139

Merged
roncodes merged 4 commits into
mainfrom
dev-v0.3.38
Jun 23, 2026
Merged

Fix sidebar navigator initial sync#139
roncodes merged 4 commits into
mainfrom
dev-v0.3.38

Conversation

@roncodes

Copy link
Copy Markdown
Member

Summary

  • simplify the sidebar navigator initial active-parent sync back to a one-time render check
  • support permission-safe navigator filtering with visible permissions and required visible children
  • fix FleetOps root-entry behavior by allowing modules to suppress initial nested sync from root URLs while preserving later route sync

Why

FleetOps can resolve the module root URL to a deeply nested route like console.fleet-ops.operations.orders.index.index. Exact route-name matching caused the sidebar to open the Operations nested menu on first module entry, which made the root sections harder to discover.

Verification

  • git diff --cached --check
  • pnpm exec ember test --filter "initial active parent predicate"

@roncodes roncodes marked this pull request as ready for review June 23, 2026 15:28
@roncodes roncodes merged commit 01e1130 into main Jun 23, 2026
7 checks passed
@roncodes roncodes deleted the dev-v0.3.38 branch June 23, 2026 15:28
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.

1 participant