-
Notifications
You must be signed in to change notification settings - Fork 289
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
💻 Class overview redesign #5610
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the new design!!!! (and hope this will not break anything....)
Thanks!! |
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
This PR introduces a new redesign for the Class Overview page, by merging it together with the 'Live Statistics' and 'Overview of programs per adventure' pages. This simplifies the UI a lot, by not confusing users having to navigate through different and uneven UI elements. In this image we can see the new UI:
In the image we can distinguish 3 different sections:
Fixes #5097
Fixes #5424
How to test
I had to open a new PR for this, but @Annelein also helped a lot in the building of this feature! Thanks a lot 😄