lit / rfcs

RFCs for changes to Lit
BSD 3-Clause "New" or "Revised" License
15 stars 10 forks source link

[RRFC] FormAssociated mixins #43

Open justinfagnani opened 4 days ago

justinfagnani commented 4 days ago

Motivation

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

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:

How

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

Current Behavior

Desired Behavior

References

justinfagnani commented 3 days ago

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 commented 3 days ago

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 commented 3 days ago

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 commented 3 days ago

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.