-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sidebar Re-design: Left Drawer #4745
Conversation
…into the main menu with a dynamic left drawer - removes pre-existing navigation components - removes teleports that swapped the navigation component - added a dynamic left drawer that can house multiple components - added a reactive mainNav component that can be interacted with across the application - refactored mobile menu toggle action - addressed menu state for situations dependent on user permissions - addressed empty state menu
Do not merge until #4744 passes review |
Please forgive the high number of changed files, the better part of them are deletions of unused components and removal of the previous teleport functionality used to manage navigation |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4745 +/- ##
=======================================
Coverage 78.79% 78.79%
=======================================
Files 311 311
Lines 14781 14781
Branches 3384 3384
=======================================
Hits 11646 11646
Misses 3135 3135
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Have made some changes here:
|
…ight margin on the nav items
Replace the pre-existing side navigation used to teleport components into the main menu with a dynamic left drawer
Description
Related Issue(s)
part of #4673
Checklist
flowforge.yml
?FlowFuse/helm
to update ConfigMap TemplateFlowFuse/CloudProject
to update values for Staging/ProductionLabels
area:migration
label