Issue #1410 Changed function name to LoadAutoScroll

[strawberries73]

Issue This PR Addresses

Fixes #1410

Type of Change

• Bugfix: Change which fixes an issue
• New Feature: Change which adds functionality
• Documentation Update: Change which improves documentation
• UI: Change which improves UI

Description

Changed the function name from LoadMoreButton to LoadAutoScroll
Changed onScroll to onClick

Checklist

• Quality: This PR builds and passes our npm test and works locally
• Tests: This PR includes thorough tests or an explanation of why it does not
• Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not (if applicable)
• Documentation: This PR includes updated/added documentation to user exposed functionality or configuration variables are added/changed or an explanation of why it does not(if applicable)

[strawberries73]

@klee214 I noticed that there is an error at onScroll on line 16. Would you like me to add it to props validation and start the PR again?

[klee214]

Sorry I didn't notice that please change the Line 55 OnClick -----> OnScroll. Also you might need to change the parent component if it use onClick instead of onScroll

[strawberries73]

@klee214 I changed the onScroll to onClick.

[humphd]

Other way around, this prop needs to be onScroll and below on the propTypes you need onScroll too. You can see how it's called at

telescope/src/frontend/src/components/Posts/Timeline.jsx

Line 48 in e987652

<LoadAutoScroll onScroll={() => nextPage()} key={`load-more-button-${pages.length}`} />

, which is sending an onScroll function.