search

icssc / AntAlmanac

A course exploration and scheduling tool for UCI Anteaters
https://antalmanac.com
MIT License
55 stars 62 forks source link

fix: remove hovered course on unmount #901

Closed KevinWu098 closed 5 months ago

KevinWu098 commented 5 months ago

Summary

  1. Set hovered course to undefined whenever SectionTableWrapped unmounts

chrome-capture-2024-1-10

Test Plan

  1. Attempt to replicate #900, by hovering a course (it should appear in the calendar), then hitting esc (or any other action that removes the course results). The hovered course should disappear.

Issues

Closes #900

js0mmer commented 5 months ago

and another (clicking the room)

https://github.com/icssc/AntAlmanac/assets/8922227/65f0a03f-cb40-4fdc-a072-51cf10278f21

js0mmer commented 5 months ago

Should also mention that bypassing via tabbing is not just limited to the back button but also added and map tabs.

https://github.com/icssc/AntAlmanac/assets/8922227/24e923bc-5ec5-4080-b5b2-0b8b21295d4a