New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove requestAnimationFrame in scrollToIndex #124

Closed

spicy-wolf opened this issue Nov 26, 2023 · 1 comment

Assignees

Labels

Owner

spicy-wolf commented Nov 26, 2023

Choices/src/Containers/ReadPage/Components/Content.tsx

Lines 120 to 124 in e90a6f1

    
           requestAnimationFrame(() => { 
        
             requestAnimationFrame(() => { 
        
               virtualizer.scrollToIndex(index, options); 
        
             }); 
        
           });

seems like it is not needed anymore

spicy-wolf added the enhancement label

spicy-wolf self-assigned this

Owner Author

spicy-wolf commented Nov 26, 2023 •

edited

Loading

nvm, still necessary TanStack/virtual#598 (comment)

spicy-wolf closed this as completed

spicy-wolf changed the title ~~remove double requestAnimationFrame in scrollToIndex~~ remove requestAnimationFrame in scrollToIndex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment