Skip to content
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

SPIKE: Test improvements to navigation #2429

Closed
nickaddy opened this issue Jun 21, 2024 · 4 comments · Fixed by #2480
Closed

SPIKE: Test improvements to navigation #2429

nickaddy opened this issue Jun 21, 2024 · 4 comments · Fixed by #2480
Assignees

Comments

@nickaddy
Copy link
Contributor

Background

As part of the Admin refresh, we'd like to experiment with bringing the left hand menus up to the top of the screen so that we can use the full width of the screen to display exercise content. We'd like to achieve a layout like this:
image.png

Feature(s) Description

Experiment with the Exercise sub menu with the aim of delivering the above configuration. Ideally, we'd like to:

  1. Have the items in the sub-menu display in a grey box underneath the top navigation
  2. Have the selected item in the sub menu appear next to the top level menu item separated by a >
  3. Have a blue line delineating which menu items are selected
  4. The menu that appears could either float on top of screen content or push the screen content down
@nickaddy
Copy link
Contributor Author

@drieJAC
Copy link
Contributor

drieJAC commented Jun 27, 2024

@drieJAC
Copy link
Contributor

drieJAC commented Jul 1, 2024

Tasks will need some logic to decide which items should be visible.
@nickaddy Draft exercises will need to be looked into at a later point.

@drieJAC
Copy link
Contributor

drieJAC commented Jul 8, 2024

We decided to remove the breadcrumbs as they will jog left and right as other menu items at the top level are selected and the breadcrumb disappears.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants