spikerj/spikersoft-issues
1
0
Fork 0
Code Issues 196 Pull Requests Actions Packages Projects Releases Wiki Activity

[Enhancement] Move Tree of Life Knowledge from Learn menu to user dropdown (below User Profile, above Admin Panel) #46

New Issue
Closed
opened 2026-05-01 02:07:34 +00:00 by spikerj · 1 comment
spikerj commented 2026-05-01 02:07:34 +00:00
Owner
Copy Link
Copy Source

Summary

Relocate Tree of Life Knowledge navigation: move it out of Learn in the menu-component hierarchy and expose it under the authenticated user dropdown instead.

Placement

  • Immediately below User Profile.
  • Immediately above Admin Panel (order: … → User Profile → Tree of Life Knowledge → Admin Panel …).
  • Only show for logged-in users consistent with sibling items in that menu (reuse same guards as Profile / Admin if applicable).

Acceptance criteria

  • Learn menu no longer contains Tree of Life Knowledge (unless another redundant link is intentional—default: remove from Learn).
  • User avatar / account dropdown lists Tree of Life Knowledge in the specified slot (under Profile, above Admin).
  • Admin Panel remains where it was relative to Profile except for inserting this one item.
  • Mobile / desktop breakpoints: item visible in folded account menu wherever Profile/Admin appear.

Implementation hint

Locate menu-component (and any shared nav config); adjust route labels + routerLink/href unchanged unless path itself moves—which is not in scope.

## Summary Relocate **Tree of Life Knowledge** navigation: move it **out of** **Learn** in the **`menu-component`** hierarchy and expose it **under the authenticated user dropdown** instead. ## Placement - **Immediately below** **User Profile**. - **Immediately above** **Admin Panel** (order: … → User Profile → **Tree of Life Knowledge** → Admin Panel …). - Only show for **logged-in users** consistent with sibling items in that menu (reuse same guards as Profile / Admin if applicable). ## Acceptance criteria - [ ] Learn menu no longer contains Tree of Life Knowledge (unless another redundant link is intentional—default: remove from Learn). - [ ] User avatar / account dropdown lists **Tree of Life Knowledge** in the specified slot (under Profile, above Admin). - [ ] **Admin Panel** remains where it was relative to Profile except for inserting this one item. - [ ] Mobile / desktop breakpoints: item visible in folded account menu wherever Profile/Admin appear. ## Implementation hint Locate **`menu-component`** (and any shared nav config); adjust route labels + `routerLink`/`href` unchanged unless path itself moves—which is **not** in scope.
spikerj commented 2026-05-07 04:41:47 +00:00
Author
Owner
Copy Link
Copy Source

Implemented

Moved Tree of Knowledge out of the Learn menu and into the user account dropdown, positioned immediately below User Profile and above Admin Panel.

Changes — menu-bar.component.html

Removed from:

  • Desktop Learn mat-menu (#learnMenu) — the @if (isLoggedIn) block containing the /tree-of-knowledge button
  • Mobile drawer Learn section — the corresponding <a> link

Added to:

  • Desktop user dropdown (#userMenu) — new mat-menu-item with routerLink="/tree-of-knowledge" and account_tree icon, placed between the User Profile button and the @if (isStaffOrAdmin()) Admin Panel block
  • Mobile account section — new <a mat-button> with the same route, icon, and (click)="onMobileNavigation()", placed in the same slot between User Profile and Admin Panel

Acceptance criteria

  • Learn menu no longer contains Tree of Knowledge
  • User avatar / account dropdown lists Tree of Knowledge below User Profile, above Admin Panel
  • Admin Panel position is unchanged (only the new item is inserted above it)
  • Mobile / desktop breakpoints: item visible in folded account menu wherever Profile/Admin appear
  • Visible for logged-in users only (it's inside the authenticated user menu, same guard scope as Profile/Admin)
## Implemented Moved **Tree of Knowledge** out of the **Learn** menu and into the **user account dropdown**, positioned immediately below **User Profile** and above **Admin Panel**. ### Changes — `menu-bar.component.html` **Removed from:** - Desktop Learn `mat-menu` (`#learnMenu`) — the `@if (isLoggedIn)` block containing the `/tree-of-knowledge` button - Mobile drawer Learn section — the corresponding `<a>` link **Added to:** - Desktop user dropdown (`#userMenu`) — new `mat-menu-item` with `routerLink="/tree-of-knowledge"` and `account_tree` icon, placed between the User Profile button and the `@if (isStaffOrAdmin())` Admin Panel block - Mobile account section — new `<a mat-button>` with the same route, icon, and `(click)="onMobileNavigation()"`, placed in the same slot between User Profile and Admin Panel ### Acceptance criteria - [x] Learn menu no longer contains Tree of Knowledge - [x] User avatar / account dropdown lists Tree of Knowledge below User Profile, above Admin Panel - [x] Admin Panel position is unchanged (only the new item is inserted above it) - [x] Mobile / desktop breakpoints: item visible in folded account menu wherever Profile/Admin appear - [x] Visible for logged-in users only (it's inside the authenticated user menu, same guard scope as Profile/Admin)
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
agentic
Actionable surfaced by the agentic architecture & security review
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 sonarqube
Code quality issue surfaced by SonarQube (bug / code smell)
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
spikerj (Joseph Spiker)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#46