Make left hand sidebar items collapsible and allow for sub-links

[jackcmeyer]

🚀 Feature Proposal

The left hand sidebar should make the items collapsible and allow for sub links.

Sidebar should look something like:

- Dashboard
- Patient
  - New Patient
  - Patient List  
- Scheduling
   - New Appointment
   - Appointment Schedule

When accessing any route /patient/**, the Patients item should be highlighted.
When accessing any route /appointments/** the Scheduling item should be highlighted.

Sub items should also be properly marked active. (E.g. when New Patient screen is shown, Patient Item should be highlighted and New Patient item should be highlighted.

Clicking on any other section should collapse the previously selected section and expand the newly selected one.

Example

It should operate very similarly to http://demo.hospitalrun.io/

Technical Notes

Should remove dropdown links from Navbar.

[anis-agwan]

Is this issue available to solve?

[sotous]

Hi! @jackcmeyer , may I have this issue assigned?

[jackcmeyer]

@sotous I have assigned this to you!

[sotous]

Thanks!

[sotous]

So I simulated the layout, nothing to fancy, just an idea and my approach to the issue, I would love your feedback @jackcmeyer in order to start working on the proposal.

Just consider the sidebar, as I said it is just an approach, so the dropdown links haven't been removed yet.

[jackcmeyer]

This looks like the right approach!

Some general feedback:

• I'm not sure the children of the selected option need to be highlighted in a lighter shade of green. I would expect only the parent and the currently selected child to be highlighted.
• We may need some sort of indication that the parents are able to be expanded and collapsed.

[sotous]

Ok I got it.

However, how should I highlight the non-selected child?, also, do you have any color suggestions to make a difference between parent and child?

[jackcmeyer]

@sotous just the default white for any selected children. I'm not sure that a color separation between the selected parent and the selected child is necessary.

[fox1t]

Here we have to hear something from @StefanoMiC too!