Skip to content

docs: improve Signal Forms complete example#69599

Open
BHUVANSH855 wants to merge 1 commit into
angular:mainfrom
BHUVANSH855:docs-signal-forms-complete-example
Open

docs: improve Signal Forms complete example#69599
BHUVANSH855 wants to merge 1 commit into
angular:mainfrom
BHUVANSH855:docs-signal-forms-complete-example

Conversation

@BHUVANSH855

Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

The "Complete example" on the Signal Forms documentation page is presented under the "Update field values with set()" section even though the example does not demonstrate value.set(). Additionally, the example does not show how to submit the form, requiring readers to refer to the later validation example to learn form submission.

Fixes #69546.

What is the new behavior?

  • Moves the "Complete example" into its own subsection so it is no longer associated with the set() section.
  • Adds form submission to the basic Signal Forms example by including:
    • a submit handler,
    • a submit button, and
    • a form submit event binding.
  • Uses a matching "Complete example" subsection under "Validation and state" for consistency.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This change is documentation-only and updates the accompanying example to better demonstrate the complete Signal Forms workflow.

@pullapprove pullapprove Bot requested a review from josephperrott July 1, 2026 04:15
@angular-robot angular-robot Bot added the area: docs Related to the documentation label Jul 1, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jul 1, 2026
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 1, 2026
@JeanMeche JeanMeche removed the request for review from josephperrott July 1, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix the “complete example” on the “Forms with signals” documentation page

2 participants