Sticky headers are broken

Question

Sticky headers are broken

Closed this issue 9 years ago · 3 comments

Because of https://github.com/gaia-components/fast-list/blob/de07ebcaa0ca7334769ec3aae3ff302b68e0c34a/fast-list.js#L45

We need to fix it without breaking variable height support.

I'll get to it, but probably not today.

Answer 1 · 2015-10-12T09:45:38.000Z

I vote we display: none the list-items that aren't required. We usually have to perform a significant reflow when updating the list any way. But I might be overlooking something :)

Answer 2 · 2015-10-12T10:12:28.000Z

My instinct was to start with visibility:hidden, but the list scrollable area will always be >= maxItemsCount * itemHeight.

If we go display:none we'll need to have a separate code path for the (legacy) "new content" support and this too.

Answer 3 · 2015-10-13T13:39:09.000Z

The code paths are actually separated enough :)
display:none it is!