avalonmediasystem / avalon

Avalon Media System – Samvera Application

http://www.avalonmediasystem.org/

Apache License 2.0

94 stars 51 forks source link

[BUG] Playlist items display past footer on playlist page #6083

Closed elynema closed 3 weeks ago

elynema commented 1 month ago

Description

On the playlist page, when there is a long list of playlist items, the box is displaying over top of the page footer. This was observed in both Chrome and Safari on desktop.

Example playlist: https://avalon-dev.dlib.indiana.edu/playlists/124

Done Looks Like

[ ] Playlist items listing matches the max height of the other Ramp elements

joncameron commented 3 weeks ago

Screenshot 2024-10-29 at 2 53 11 PM 👍 Displaying as expected.

elynema commented 3 weeks ago

@joncameron @Dananji This is not working in Chrome:

Dananji commented 3 weeks ago

It is working for me in Chrome. Maybe it needs a hard reload?

elynema commented 3 weeks ago

@Dananji Can't believe my typo. Yes, it is working in Chrome. It is not working in Safari. I apologize for the mistake.

Dananji commented 3 weeks ago

On first page load the playlist items list was overflowing footer on a normal browser window. I checked the CSS to make sure it has the code change I made, and it was still on the old CSS. But opening it up on a private browser window seems to be displaying it with the latest CSS changes.

@elynema Could you please try this on a private browser window to see whether you are able to see the same results?

elynema commented 3 weeks ago

@Dananji Yes, looks good now on all devices! I will close this.