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

Comment: Hmm I understand that we need to add this here because at the form cmp level these are not loaded as we can add new fields dynamically. #597

Open
kpsherva opened this issue Mar 4, 2024 · 0 comments

Comments

@kpsherva
Copy link
Contributor

kpsherva commented Mar 4, 2024

          Comment: Hmm I understand that we need to add this here because at the form cmp level these are not loaded as we can add new fields dynamically.

The only issue I can see is that in terms of performance this is worse as it will loop over items that were already processed. I would rather set the onkeydown when creating it, but in a first look it doesn't seem easy to do with rjsf ... However it's probably kicking in only when working on array fields so it should be good enough IMHO.

Originally posted by @jrcastro2 in #593 (comment)

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

No branches or pull requests

1 participant