icssc / peterportal-client

A web application to aid UCI students with course discovery and planning.
https://peterportal.org
MIT License
16 stars 14 forks source link

Roadmap popovers overflow #373

Closed js0mmer closed 9 months ago

js0mmer commented 1 year ago

https://github.com/icssc/peterportal-client/assets/8922227/0c7c6985-11ae-425f-9529-a47648f22b6e

image

timobraz commented 11 months ago

I'm thinking of solving this issue by hiding this popup anytime the user scrolls in the div of all the years. This behavior is natural with other websites where scrolling hides popups automatically.

js0mmer commented 10 months ago

Better approach may be to add a container attribute to the overlay trigger. Should be possible then to set an appropriate z-index and hide the overflow so the popover is not visible outside of the planner. Blocked by #167