svelte-virtual-list icon indicating copy to clipboard operation
svelte-virtual-list copied to clipboard

#43 scroll position

Open gitbreaker222 opened this issue 4 years ago • 7 comments

Before

Various use cases exist, where the scroll-position is not as expected - closes #43

After

A new method scrollToIndex is exposed to control the scroll-position from the consumer

This resolves following issues and PRs:

closes #15 closes #28 closes #30 closes #31 (PR) closes #32 closes #60 (PR; similar to #15)

gitbreaker222 avatar Jan 23 '21 15:01 gitbreaker222

looking at #44 I think I should keep the height map part and test with variable-height-items :thinking:

edit: tested again and removed the fixed height: with and without the height-map part the scrollToItem method is unreliable. I think it is best to solve these things step by step:

  • scroll-position for lists with fixed-height-items
  • height-map for variable height items

gitbreaker222 avatar Jan 23 '21 22:01 gitbreaker222

Hi! is this available already?

BlueBomberman avatar Jun 09 '21 08:06 BlueBomberman

@BlueBomberman Unfortunately I haven't got any response from Rich Harris on this since about 6 months. Maybe I can ping somebody else from the svelte team? Let's find out:

@Conduitry are you maybe able to review/merge/assign this PR? It would resolve 5 issues 1 other PR 🌟

edit 2021-06-18 or @pngwn maybe?

gitbreaker222 avatar Jun 09 '21 12:06 gitbreaker222

@gitbreaker222 I suppose to make a fork will be much better in its current state.

stalkerg avatar Jul 05 '21 07:07 stalkerg

I belive this may also fixes that if items are changed from the consumer (items filtered) the scroll position may out of range leaving the canvas empty.

juancarrey avatar Aug 11 '21 12:08 juancarrey

I would really love to see this PR merged if it's possible. Is there any way we can help this component going forward? Are you looking for active maintainers etc?

mellson avatar Jan 22 '22 10:01 mellson

community-edition package finally available. Thx again @mellson and sry for the long wait everyone https://www.npmjs.com/package/svelte-virtual-list-ce

There will be some more upcoming changes to the readme and other maintenance stuff maybe :)

gitbreaker222 avatar Jan 23 '22 12:01 gitbreaker222