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

Add (loaded) event #237

Open
yktoo opened this issue Jul 25, 2021 · 2 comments
Open

Add (loaded) event #237

yktoo opened this issue Jul 25, 2021 · 2 comments

Comments

@yktoo
Copy link

yktoo commented Jul 25, 2021

Thanks for the great component!

It'd be very nice to have some callback, e.g. (loaded) event, in <re-captcha>, without needing to create a customized RecaptchaLoaderService.

Loading the CAPTCHA may take a while, especially on a slow connection, so it'd make sense to display a spinner in the containing element. That spinner could then be removed in the proposed (loaded) event handler.

@DethAriel
Copy link
Owner

Hi @yktoo ! You are absolutely right, having such an event would certainly be a useful addition to the component API. Maybe even include a RecaptchaLoaderService.preload() method for potential onHover preloading 🤔

I'll look into it, thank you for the suggestion!

@pavandixit93
Copy link

It would be really nice feature, I also need loaded event. I want to add transform style on fully rendered dynamic captcha but since i don't know when captcha is rendered i am unable to call my function which will reset transform on captcha load.
Currently I am using javascript setTimeout function with some timeout and inside that i am calling my transform function.

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

No branches or pull requests

3 participants