Skip to content

Refreshing the forms pages in the cookbook - #13957

Merged
sfshaza2 merged 1 commit into
mainfrom
cookbook-forms
Oct 6, 2026
Merged

sfshaza2 merged 1 commit into
mainfrom
cookbook-forms

Conversation

@sfshaza2

@sfshaza2 sfshaza2 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Just some general dusting and cleaning and updateing to some of our current docs standards.

@sfshaza2
sfshaza2 requested a review from a team as a code owner October 5, 2026 18:00

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates several Flutter cookbook examples and documentation files related to forms. Key changes include declaring FocusNode as late final, using private state classes with arrow syntax for createState(), simplifying widget lists, fixing typos, and updating image formats. Feedback on the documentation changes points out an incorrect reference to a non-existent API value, AutovalidateMode.onUnfocus, in a newly added tip block.

Comment thread sites/docs/src/content/cookbook/forms/validation.md
@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit f859352):

https://flutter-docs-prod--docs-pr13957-cookbook-forms-sipu7kv9.web.app

@justinmc justinmc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread sites/docs/src/content/cookbook/forms/validation.md

@Renzo-Olivares Renzo-Olivares left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM w/ some small comments.

Comment thread sites/docs/src/content/cookbook/forms/focus.md
Comment thread sites/docs/src/content/cookbook/forms/retrieve-input.md
Comment thread sites/docs/src/content/cookbook/forms/text-field-changes.md
Comment thread sites/docs/src/content/cookbook/forms/text-field-changes.md

@conooi conooi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@sfshaza2
sfshaza2 merged commit 9fa5960 into main Oct 6, 2026
16 checks passed
@sfshaza2
sfshaza2 deleted the cookbook-forms branch October 6, 2026 02:56
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.

5 participants