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

Support arguments for FACE callbacks #4093

Closed
nolanlawson opened this issue Mar 22, 2024 · 0 comments · Fixed by #4094
Closed

Support arguments for FACE callbacks #4093

nolanlawson opened this issue Mar 22, 2024 · 0 comments · Fixed by #4094

Comments

@nolanlawson
Copy link
Contributor

Several of the FACE (Form Associated Custom Element) callbacks have function arguments:

  • formAssociatedCallback(form)
  • formDisabledCallback(disabled)
  • formStateRestoreCallback(state, mode)

However, currently, LWC ignores the arguments and just calls the function without any arguments:

if (isTrue(nativeLifecycleElementsToUpgradedByLWC.get(this))) {
callback(this);
}

We should support the arguments for these methods.

@nolanlawson nolanlawson added enhancement Up for grabs Issues that are relatively small, self-contained, and ready for implementation and removed Up for grabs Issues that are relatively small, self-contained, and ready for implementation labels Mar 22, 2024
nolanlawson added a commit that referenced this issue Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant