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

JS errors when text field is used in a grid and disabled #530

Closed
hinder1 opened this issue Dec 16, 2024 · 2 comments · Fixed by #532
Closed

JS errors when text field is used in a grid and disabled #530

hinder1 opened this issue Dec 16, 2024 · 2 comments · Fixed by #532
Assignees
Labels
bug Something isn't working

Comments

@hinder1
Copy link

hinder1 commented Dec 16, 2024

Hi,
We use SuperBigDecimalField in a grid.
If the SuperBigDecimalField is first disabled and then enabled, we got some JS errors.
This is because the delayed initialization (SuperTextField#performDelayedInitialization) isn't executed as the field is disabled and the communication from the client to the server is blocked.
Is it possible to set @ClientCallable(DisabledUpdateMode.ALWAYS) to SuperTextField#performDelayedInitialization so that the method is always executed?

superfields 0.19.1
Vaadin 24.5.0
GridError.zip

@vaadin-miki
Copy link
Owner

hello @hinder1 and thanks for using superfields

and thanks for a perfect bug ticket, together with reproducable code :) the fix will be added to the next maintenance release, I hope to have time to do it this week still

@vaadin-miki vaadin-miki self-assigned this Dec 17, 2024
@vaadin-miki vaadin-miki added the bug Something isn't working label Dec 17, 2024
@vaadin-miki vaadin-miki added this to the Next release milestone Dec 17, 2024
@vaadin-miki vaadin-miki changed the title JS errors when used in a grid and is disabled first JS errors when text field is used in a grid and disabled Dec 17, 2024
vaadin-miki added a commit that referenced this issue Dec 17, 2024
@vaadin-miki vaadin-miki mentioned this issue Dec 17, 2024
vaadin-miki added a commit that referenced this issue Dec 17, 2024
vaadin-miki added a commit that referenced this issue Dec 17, 2024
@vaadin-miki
Copy link
Owner

@hinder1 due to some unexpected gpg error the release is not yet in Maven and I do not have time to look into it today - in the mean time you could use github packages; I will try to get the thing sorted before the year ends - sorry for the trouble

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants