statsfm / web

stats.fm web repository
https://stats.fm
GNU General Public License v3.0
115 stars 26 forks source link

Image Grid #61

Open CRortyDG opened 4 months ago

CRortyDG commented 4 months ago

When you expand the grid of a 'Top' list, the grid displays one less column than the top row.

The carousel row continues falling off the page, and this is causing a duplicate first wrapped item.

image

The grid and the scroll able carousel are two different items of complete lists, currently the grid list is shifted to overlap its top row. This can be fixed by hiding the carousel when the grid is displayed and undoing the transform: translateY(-278px); and other css that forces the grid to overlap with the carousel.

image

when we remove the css we see the two full lists image

Then hiding the carousel gives us our desired view -

image

CRortyDG commented 4 months ago

This also provides cleaner functionality (imo) when using the pagination as the row only shifts up, instead of both the carousel and row shifting.