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 redesign #843

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Sidebar redesign #843

wants to merge 4 commits into from

Conversation

jvmespark
Copy link
Collaborator

Issue
closes #838

Photos
sidebar-redesign

Additional Info
Impact of changes:

  • Gave the user more room to breathe by simplifying the sidebar
  • Optimized for the most common case: scrolling through courses
  • Reduced filters down to a modal because it's not used often enough to have to always be on display
  • Opened up new opportunities for developers to make course filters by giving it its own dedicated space

@jvmespark
Copy link
Collaborator Author

some older ideas that we can introduce to the new modal:
#777
#500

@jvmespark jvmespark added the Review Ready! Inform team that PR is ready for review label Nov 11, 2023
@EggyRepublic
Copy link
Collaborator

I think filter by department should be its own button due to how common its likely to be used. The dropdown could also be broken down for easier navigation like such:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review Ready! Inform team that PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request — Side bar redesign
2 participants