Skip to content
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

Page input debounce #53

Closed
NadavShaar opened this issue Jan 20, 2021 · 1 comment
Closed

Page input debounce #53

NadavShaar opened this issue Jan 20, 2021 · 1 comment
Assignees
@ilayofir
Labels
wontfix This will not be worked on
Projects
v0.2.0
Milestone
Bugs & Nuggets

Comments

@NadavShaar
Copy link
Owner

NadavShaar commented Jan 20, 2021 •
edited
Loading

Add debounce for the page input so if the user would like to navigate to page 32 for example, it won't navigate to page 3 first.
@NadavShaar NadavShaar added bug Something isn't working enhancement New feature or request labels Jan 20, 2021
@NadavShaar NadavShaar added this to TO-DO in v0.2.0 Jan 20, 2021
@NadavShaar NadavShaar added this to the Bugs & Nuggets milestone Jan 22, 2021
@ilayofir
Copy link
Collaborator

Async flows have their own debounce functionality already built in. So, this is relevant only for sync flow, which IMO should not require any debounce functionality, as it will reduce user experience.
Closing this for now.

@NadavShaar NadavShaar added wontfix This will not be worked on and removed bug Something isn't working enhancement New feature or request labels Jan 30, 2021
@NadavShaar NadavShaar moved this from TO-DO to DONE in v0.2.0 Jan 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ilayofir ilayofir

Labels
wontfix This will not be worked on
Projects
No open projects
v0.2.0
DONE
Milestone
Bugs & Nuggets
Development

No branches or pull requests
2 participants
@NadavShaar @ilayofir