search

Western-Rank / western-rank

A course review platform for Western University and affiliate colleges!
https://westernrank.com
4 stars 0 forks source link

Professor Preview for each course #70

Open ArsalaanAli opened 1 year ago

ArsalaanAli commented 1 year ago

In the course information section, show the 3 most reviewed profs (recently) and show their rmp score and num of reviews

basokant commented 1 year ago

I'd prefer having the professor preview card be shown as a popover/tooltip, where the user would hover over the professor name under a review and in the course info section to see this RMP card.

If you truly think that it'd be useful for people and worth the space, then I think we should go ahead with it! If we were to have those top 3 professor cards, then there are a few things that I'd personally want to see changed (but obviously these things are up to discussion!!):

  1. Move the professor cards to the section w Requisites, locations, and extra info. Where you put them now I think is taking up too much vertical space, making it harder to find the written reviews and course info.
  2. Consider having less info on the cards if they're always showing. Alternatively we could consider having a compressed view of the professor card that only had the professor name and overall rating, and then when collapsed it could show the rest of the info.
  3. Use the stars component for anything that's not a percentage.
  4. Nit, but I think we should adjust some styles (layout, sizing, colours) that I don't want to put into words rn but I'll get back to u later on.