Skip to content

collapse for RMP data#72

Open
GiangNPham wants to merge 1 commit intomasterfrom
feature/dropdown_rmp
Open

collapse for RMP data#72
GiangNPham wants to merge 1 commit intomasterfrom
feature/dropdown_rmp

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.

2 participants