Skip to content

HOLD: Use "Submit" label on /new form buttons#1863

Draft
maebeale wants to merge 2 commits into
mainfrom
maebeale/new-form-submit-label
Draft

HOLD: Use "Submit" label on /new form buttons#1863
maebeale wants to merge 2 commits into
mainfrom
maebeale/new-form-submit-label

Conversation

@maebeale

Copy link
Copy Markdown
Collaborator

What is the goal of this PR and why is this important?

  • Standardizes the submit call to action on new-record forms to read "Submit" instead of "Create <Model>"
  • Gives creation screens a consistent label across the app

How did you approach the change?

  • Replaced explicit "Create …" submit labels with "Submit" on forms served from /new endpoints
  • For partials shared between new and edit, only the new-record branch was changed; the edit label (e.g. "Save changes" / "Update") is preserved
  • Files touched:
    • forms/new.html.erb — "Create form" → "Submit"
    • payments/_form.html.erb (rendered only by payments/new) — "Create Payment" → "Submit"
    • allocations/new.html.erb — "Create Allocation" → "Submit"
    • refunds/new.html.erb — "Create Refund" → "Submit" (kept "Refund via Stripe" branch)
    • scholarships/_form.html.erb — new branch "Create scholarship" → "Submit"
    • event_registrations/_form.html.erb — new branch "Create registration" → "Submit"

Anything else to add?

  • Marked HOLD pending confirmation on scope.
  • Intentionally left unchanged (flagging for reviewer):
    • payments/_allocation_form.html.erb and discounts/_allocation_form.html.erb ("Create Payment"/"Create Discount") render via the custom allocation_form turbo action, not a /new route.
    • ~30 forms use bare f.button :submit (no explicit label); simple_form auto-renders these as "Create <Model>" on new / "Update <Model>" on edit. Switching those to "Submit" would require a per-form persisted? conditional in each shared partial — held off pending direction.

🤖 Generated with Claude Code

maebeale and others added 2 commits June 22, 2026 08:30
Replaces "Create <Model>" submit labels with "Submit" on new-record
forms so the call to action is consistent across creation screens.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Catches the create-branch label introduced on main so all new-record
forms read "Submit" consistently.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
<div class="ml-auto flex items-center gap-3">
<%= link_to "Cancel", cancel_path, class: "btn btn-secondary-outline" %>
<%= f.button :submit, f.object.persisted? ? "Save changes" : "Create registration", class: "btn btn-primary" %>
<%= f.button :submit, f.object.persisted? ? "Save changes" : "Submit", class: "btn btn-primary" %>

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 From Claude: This partial is shared by new and edit, so only the non-persisted branch changes to "Submit" — the edit label ("Save changes") is intentionally preserved.

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

Successfully merging this pull request may close these issues.

1 participant