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

Sidebar #10

Merged
merged 27 commits into from
Nov 29, 2024
Merged

Sidebar #10

merged 27 commits into from
Nov 29, 2024

Conversation

niclasheun
Copy link
Contributor

**Important: This PR depends on #8 **

Description

  • Added Sidebar to management console
  • Added user display with gravatar image
  • Added Dark Mode Toggle and DarkModeThemeProvider

Known Issues

  • The sidebar is currently still filled with demo data. To keep PR rather 'small', I will replace the demo data in a new PR.

Screenshots

image image

@niclasheun niclasheun self-assigned this Nov 26, 2024
@niclasheun niclasheun marked this pull request as draft November 26, 2024 21:47
@niclasheun
Copy link
Contributor Author

Optisch noch etwas hässlich, aber jetzt funktioniert es zumindest mal.
Problem ist, dass Shadn in der Mobile version überhaupt nicht die zwei Layer sidebar unterstützt (zumindest nicht out of the box).

Screenshorts:

image image image

@niclasheun
Copy link
Contributor Author

As this PR has already become way to big:

Main features

  • Double sidebar, working on mobile and desktop (-> see video)
  • Routes are dynamically imported from submodules
  • Each course icon and color are loaded from the course meta data (if not existent, fallback to default values)

Open Todos (-> next PR)

  • Export 2nd sidebar elements from the course phase modules
  • Import these sidebar elements and display them
  • Update the breadcrums to automatically show the right location based on the url location

@niclasheun niclasheun marked this pull request as ready for review November 28, 2024 09:14
@niclasheun niclasheun merged commit 7a1b0ed into main Nov 29, 2024
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.

2 participants