Skip to content

feat: Reorganize Knowledge taxonomy and enhance collapsed sidebar navigation#283

Open
esafwan wants to merge 6 commits into
developfrom
feat/sidebar-two-level-nav
Open

feat: Reorganize Knowledge taxonomy and enhance collapsed sidebar navigation#283
esafwan wants to merge 6 commits into
developfrom
feat/sidebar-two-level-nav

Conversation

@esafwan
Copy link
Copy Markdown
Contributor

@esafwan esafwan commented Jun 2, 2026

Motivation

The sidebar navigation logic was complex and fragmented, resulting in inconsistent layouts across different application domains (Data, Knowledge, Models, etc.). This PR standardizes the sidebar by adopting a top-level unified navigation structure using Shadcn UI's Sidebar layout.

Changes

  1. Unified Sidebar Layout: Reverted the fragmented navigation patterns to a cohesive <UnifiedLayout> component.
  2. Accordion Navigation: The primary sidebar now uses a unified accordion logic allowing only one sub-menu to be open at a time for better UX.
  3. Collapsed Sidebar Flyouts: When the sidebar is collapsed into a narrow rail, hovering over or clicking a sidebar item displays a clean flyout popover (using DropdownMenu), making navigation fully accessible without awkward rail expansion.
  4. Action-Based Taxonomy: The 'Knowledge' sub-items were refactored based on user actions:
    • Tables: For structured/relational data building.
    • Documents: For unstructured document retrieval (RAG).
  5. Action Bar Regeneration: Fixed the action buttons (Create/Add) regression within Data and Knowledge pages by properly passing headerActions into the UnifiedLayout.

This resolves the layout jumping, ensures high UX standards, and provides a sleek interaction model reminiscent of modern apps like Linear.

esafwan added 6 commits June 2, 2026 05:13
Refactors the app layout to use a secondary internal sidebar for Knowledge and Settings pages.
Reverts the broken InnerSidebarLayout approach and implements proper in-sidebar sub-navigation using ShadCN Collapsible pattern for Knowledge and Settings pages.
@esafwan esafwan changed the title feat: Implement App-Wide Sidebar with Two-Level Navigation & Flyout feat: Reorganize Knowledge taxonomy and enhance collapsed sidebar navigation Jun 2, 2026
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