Skip to content

Conversation

@GiangNPham
Copy link
Contributor

No description provided.

@axdyv
Copy link
Contributor

axdyv commented Apr 10, 2025

This is a great start but here are some things to consider:

  • Instead of having a more details button, can we just make the arrow next to professor details to reduce redundancy?
  • Also we want users to not have to hide the drop down everytime, so when a user clicks another class, the site remembers to keep the dropdown hidden.

The functionality looks solid, though. If you want to see more feedback, you can check the general channel on the Dev Discord. I consolidated the most important points, though.

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.

3 participants