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

[RRFC] FormAssociated mixins #43

Open
1 task done
justinfagnani opened this issue Sep 20, 2024 · 5 comments
Open
1 task done

[RRFC] FormAssociated mixins #43

justinfagnani opened this issue Sep 20, 2024 · 5 comments

Comments

@justinfagnani
Copy link
Contributor

justinfagnani commented Sep 20, 2024

  • I searched for an existing RRFC which might be relevant to my RRFC

Motivation

Building a form associated custom element requires a lot of common boilerplate:

  • Setting static formAssociated = true
  • Attaching ElementInternals
  • Setting a default ARIA role
  • Calling internals.setFormValue() when needed
  • Representing the element's form value and the element state
  • Exposing the value to the element's public API
  • Handling formDisabledCallback()
  • Handling formResetCallback() and formStateRestoreCallback()

Example

A simple-ish FormAssociated() element, with validation, might just forward an input's value to the element's value:

import {FormAssociated, value} from '@lit-labs/form';

class MyFormElement extends FormAssociated(LitElement, {
  role: 'textbox'
}) {

  @formValue()
  accessor value: string = '';
  
  @query()
  accessor #input: HTMLInputElement;

  render() {
    return html`<input .value=${this.value}> @input=${this.#onInput}`;
  }

  #onInput(e) {
    this.value = e.target.value;
  }

  protected _checkValidity() {
    if (this.value.length < 5) {
      return {
        flags: { tooLong: true },
        message: "value is too long",
        anchor: this.#input,
      };
    }
  }
}

In this case the FormAssociated mixin would:

  • set static formAssociated = true
  • set internals.role
  • call setFormValue() when value changes
  • Implements formStateRestoreCallback() to reset the value.

How

A FormAssociated() mixin, paired with @internals, @formValue and @formState decorators.

Current Behavior

Desired Behavior

References

  • n/a
@justinfagnani
Copy link
Contributor Author

There are a lot of possible variations on what a mixing might provide, especially in terms of the public API it creates on the element.

I think the the minimal end of the spectrum, like the sketch above, a mixin might add no new public API. It just helps the element author coordinate the form value, form associated callbacks, and form methods on ElementInternals.

Another point on the spectrum is to emulate many of the the methods of HTMLInputElement, like checkValidity(), validity, etc. HTMLInputElement is also very customizable in terms of validity constraints. I don't think most elements will want the element users to customize the constraints, but the element author may want to use a similar declarative API (min, max, pattern, etc) to define constraints, rather than have just a validity checking callback.

So we may want to start with a minimal-API-adding mixin, then layer a richer API mixin on top of that. The minimal-API mixin can optionally have support for pluggable constraints.

@maxpatiiuk
Copy link

I think the the minimal end of the spectrum, like the sketch above, a mixin might add no new public API. It just helps the element author coordinate the form value, form associated callbacks, and form methods on ElementInternals.

If that is the case, could FormAssociated be developed as a controller instead?
That would provide more flexibility and (marginally?) better performance.

@justinfagnani
Copy link
Contributor Author

If that is the case, could FormAssociated be developed as a controller instead?

No, because it needs to override the form associated instance methods like formResetCallback().

@jpzwarte
Copy link

I love this. One thing though that should be considered & documented carefully is how to make such a custom form control accessible.

I'm not sure browsers are fully there yet for formAssociated controls.

@JamesIves
Copy link

Supportive of this. Working with forms is fairly confusing, so utilities like this are really helpful.

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

4 participants