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.

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

feat: improve UX when user is on a slow network #70

Merged
merged 5 commits into from
Nov 9, 2024

Conversation

CAMOBAP
Copy link
Collaborator

@CAMOBAP CAMOBAP commented Nov 3, 2024

Related issue #64

  • add closableLoading property to allow user to tap to cancel hCaptcha during loading
  • hasBackdrop=false now affects backgroundColor and no background will be shown till hCaptcha is loaded
  • emit 'error' if hCaptha wasn't load in 15s

closes #65

- add `closableLoading` property to allow cancel hCaptcha during loading
- `hasBackdrop=false` now affect `backgroundColor` and no
  backgroun will be shown till hCaptcha is loaded
@CAMOBAP CAMOBAP requested a review from e271828- November 3, 2024 20:57
@CAMOBAP CAMOBAP self-assigned this Nov 3, 2024
@e271828- e271828- changed the title feat: improve slow network experience feat: improve UX when user is on a slow network Nov 3, 2024
index.js Show resolved Hide resolved
@e271828-
Copy link
Contributor

e271828- commented Nov 3, 2024

looks good at first glance, need README update + ver bump

@CAMOBAP CAMOBAP force-pushed the feature/improve-slow-network-experience branch from 1b8c785 to 21742cc Compare November 3, 2024 21:31
@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Nov 3, 2024

looks good at first glance, need README update + ver bump

I did manual tests for all combinations of hasBackdrop, showLoading, and closableLoading no issue was spotted

@CAMOBAP CAMOBAP requested a review from e271828- November 4, 2024 08:26
@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Nov 4, 2024

README updated

@CAMOBAP CAMOBAP merged commit e329552 into master Nov 9, 2024
7 checks passed
@dev2xl
Copy link

dev2xl commented Nov 28, 2024

I've been looking forward to this! Do you know when it will be released on npm?

One suggestion why not letting the user specify the timeout, while still having the 15 sec as default?

@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Nov 28, 2024

@dev2xl please check 1.8.2 release, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backdrop makes the app unusable if there is no internet connection
3 participants