Specify form selector

[julianrubisch]

Enhancement

Description

This PR adds the possibility to declare a custom form selector to be used when scooping up form parameters:

<div data-reflex="..." data-reflex-form-selector="#custom_form">....</div>

this.stimulate("MyReflex#action", this.element, { formSelector: "#custom_form" })

Note: This is based on my work on #517, the actual diff is much smaller: 4c39aff

Why should this be added

At the moment I'm building a WizardReflex where the actual form is separated from the navigation buttons/tabs, thus form params wouldn't be submitted.

Checklist

• My code follows the style guidelines of this project
• Checks (StandardRB & Prettier-Standard) are passing
• This is not a documentation update

Please note that the best way to suggest changes or updates to the documentation is to join Discord and leave a note in the #docs channel. Any documentation updates posted as PRs cannot be accepted at this time. ❤️

[leastbad]

Out of the gate: there are documentation changes in here. Please, please take them out.

[julianrubisch]

easy, this is going to rebased once #517 lands

[leastbad]

I can't stress enough how poorly GitBook handles non-clean merges. I get super jumpy when I see things creep in...

[marcoroth]

I like the general direction of this PR 👍🏼

[julianrubisch]

I can't stress enough how poorly GitBook handles non-clean merges. I get super jumpy when I see things creep in...

Admittedly, it's hard to reason about it this way. Still I wanted to put it out there. We'll have to wait for @hopsoft to review #517

[leastbad]

Essentially, every paragraph in the docs that has at least one changed character gets duplicated 2-3 times, showing a before/after... but it's all just text. You have to scan every page looking for dupes, and fussing to make sure the right version persists.

It takes hours.

[hopsoft]

Nice work. Let's wait for #517 to land then get this merged.