Skip to content

Clarify ListTile error reporting in colored widget#13222

Open
Albert221 wants to merge 3 commits intoflutter:mainfrom
Albert221:patch-2
Open

Clarify ListTile error reporting in colored widget#13222
Albert221 wants to merge 3 commits intoflutter:mainfrom
Albert221:patch-2

Conversation

@Albert221
Copy link
Copy Markdown
Contributor

Description of what this PR is changing or adding, and why:

My colleagues (and I) had a heart attack when they saw an exception would be (supposedly) thrown in such a case until they dug deeper into the article's contents.

Updated the title and description to clarify that ListTile reports an error in debug mode when wrapped in a colored widget instead of throwing an exception.

Issues fixed by this PR (if any):

None

PRs or commits this PR depends on (if any):

None

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

Updated the title and description to clarify that ListTile reports an error in debug mode when wrapped in a colored widget instead of throwing an exception.
@Albert221 Albert221 requested review from a team and sfshaza2 as code owners March 26, 2026 15:16
Copy link
Copy Markdown
Contributor

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

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 the documentation for a breaking change regarding ListTile to clarify that it reports an error in debug mode instead of throwing a general exception. The reviewer noted that the title change should also be reflected in the main breaking changes index file for consistency.

Copy link
Copy Markdown
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm

@sfshaza2
Copy link
Copy Markdown
Contributor

@Albert221, please resolve the conflict(s).

@flutter-website-bot
Copy link
Copy Markdown
Collaborator

Visit the preview URL for this PR (updated for commit c24b294):

https://flutter-docs-prod--pr13222-patch-2-jcx37m5e.web.app

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.

3 participants