Closed fozziethebeat closed 1 year ago
@fozziethebeat I think I could take this issue. Could you assign it to me? Self-assigning needs Github's Collaborator role i think...
Great!@ Yeah, the assigning part is a bit of a hassle. After the MVP we might want to have a group discussion about simplifying it.
Also, since this is quite a few changes, I'd request this be done in a couple PRs rather than one giant PR.
could this be closed?
Right now
website/src/pages/dashboard
includes a niceSidebar
with multiple menu options and then presents the select page's contents.This is a great design that would be nice to use in other scenarios such as for an admin specific page which would have different sidebar options.
To do this I think we need to:
website/src/components/Dashboard/SideMenu
it's own independent component that takes in a list of menu options as a property.website/src/pages/dashboard
into a layout component inwebsite/src/components/SideMenuLayout
that also takes in the list of menu options.GeneralSidebarLayout
andAdminSidebarLayout
, and maybe others.GeneralSidebarLayout
inwebsite/src/pages/dashboard
AdminSidebarLayout
in future admin pages but using a different color scheme and different sidebar options.