Support data-turbo-action="..." on Form Submits

[seanpdoyle]

The Navigator class translates <form method="get"> submissions into
proposeVisit calls, skipping the rest of the FormSubmission
architecture.

This commit adds support for declaring [data-turbo-action="replace"]
(or "restore" or "advance") as a Visit Action. Prior to this
commit, specifying a Visit action was limited to an <a>
click-initiated Visit only.

This can be useful for a GET form driven by auto-submission driving
the current page's URL, like a Typeahead autocomplete <input type="search"> field or a bounding-box scrolling Map. In those
scenarios, a series of "advance" page visits would yield an
essentially broken "Back" and "Forward" navigation experience.

[sstephenson]

Excellent!

[olliekav]

@seanpdoyle Apologies for posting this on an old merge, but should this make the following work?

<%= form_for(@event, url: events_path, html: { data: { "turbo-action": "replace" }}) do |f| %>

or

<%= button_to event_path(@event), method: :delete, data: { confirm: "Delete this event?" }, form: { "data-turbo-action": "replace" } do %>

I'm trying to get this to fire on Turbo-iOS but I'm always getting an .advance

[lutzcc1]

@seanpdoyle Apologies for posting this on an old merge, but should this make the following work?

<%= form_for(@event, url: events_path, html: { data: { "turbo-action": "replace" }}) do |f| %>

or

<%= button_to event_path(@event), method: :delete, data: { confirm: "Delete this event?" }, form: { "data-turbo-action": "replace" } do %>

I'm trying to get this to fire on Turbo-iOS but I'm always getting an .advance

Hey @olliekav, were you able to make this work? I'm also trying to achieve a replace action on form submission to avoid the advance animation in turbo-ios

[olliekav]

@lutzcc1 Yep, it's working for me. I had to set a resolution in my package.json to a forked version of @hotwired/turbo though.

"resolutions": {
    "@hotwired/turbo": "https://github.com/olliekav/turbo#form-turbo-action"
  }

This has now been merged into the main branch now, so you can use that... #409