search

petyosi / react-virtuoso

The most powerful virtual list component for React
https://virtuoso.dev
MIT License
5.25k stars 301 forks source link

TopItemListContainer must be part of the custom components #332

Closed ezsper closed 3 years ago

ezsper commented 3 years ago

I'm facing a problem with TopItemListContainer that when using groups it doesn't allow us to set a proper z-index. A proper solution would be to make TopItemListContainer a customisable component as Item or Group. Currently I'm having to apply zIndex with this selector '& div[style*="position: sticky"]': { zIndex: 999 }

petyosi commented 3 years ago

Agreed, that's a nice idea. Why don't you open a PR? That part of the code is fairly trivial.

ezsper commented 3 years ago

Agreed, that's a nice idea. Why don't you open a PR? That part of the code is fairly trivial.

https://github.com/petyosi/react-virtuoso/pull/335

ezsper commented 3 years ago

Merged.