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

Learning Analytics / Main Menu Redesign #867

Closed
ggurdin opened this issue Oct 29, 2024 · 6 comments · Fixed by #895
Closed

Learning Analytics / Main Menu Redesign #867

ggurdin opened this issue Oct 29, 2024 · 6 comments · Fixed by #895
Assignees

Comments

@ggurdin
Copy link
Collaborator

ggurdin commented Oct 29, 2024

20241029_121059

@linhtphung
Copy link

A small tweak before a full design is done:

  • Remove the avatar icon below the search box because there's already one in the search box.
  • Make the Level bar shorter and then move it next to the Lemmas and Grammar icons so that everything is one one line
Screenshot 2024-10-30 092607

@ggurdin
Copy link
Collaborator Author

ggurdin commented Oct 31, 2024

Initial frontend work, on mobile on web:
Screenshot 2024-10-31 at 12 31 10 PM
Screenshot 2024-10-31 at 12 32 45 PM

@ggurdin
Copy link
Collaborator Author

ggurdin commented Nov 1, 2024

padding between analytics and search bar

@KhueDao1 KhueDao1 self-assigned this Nov 1, 2024
@KhueDao1
Copy link
Collaborator

KhueDao1 commented Nov 1, 2024

I will prototype some variations for better attention hierarchy on Wednesday @KhueDao1

@ggurdin ggurdin linked a pull request Nov 1, 2024 that will close this issue
@ggurdin
Copy link
Collaborator Author

ggurdin commented Nov 1, 2024

I'm closing this because I've merged in the initial design.
This design will probably change between now and the next release, so I will leave out the 'to test' instructions on this ticket

@ggurdin
Copy link
Collaborator Author

ggurdin commented Nov 5, 2024

To test:

  • Look at the menu on each platform and make sure it looks as expected

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 a pull request may close this issue.

3 participants