Skip to content

Fix doc typo in WidgetsApp.supportedLocales#51646

Merged
shihaohong merged 2 commits intoflutter:masterfrom
shihaohong:locale-doc-fix
Mar 2, 2020
Merged

Fix doc typo in WidgetsApp.supportedLocales#51646
shihaohong merged 2 commits intoflutter:masterfrom
shihaohong:locale-doc-fix

Conversation

@shihaohong
Copy link
Copy Markdown
Contributor

Description

Simple doc fix in WidgetsApp.supportedLocales.

@shihaohong shihaohong added framework flutter/packages/flutter repository. See also f: labels. d: api docs Issues with https://api.flutter.dev/ labels Feb 28, 2020
@fluttergithubbot
Copy link
Copy Markdown
Contributor

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@shihaohong shihaohong changed the title Fix doc typo in LocaleInfo Fix doc typo in WidgetsApp.supportedLocales Feb 28, 2020
@HansMuller
Copy link
Copy Markdown
Contributor

I think the doc is OK (it appears to render correctly - https://master-api.flutter.dev/flutter/widgets/WidgetsApp/supportedLocales.html), markdown for numbered list items just has to show a leading number; any number.

Copy link
Copy Markdown
Contributor

@HansMuller HansMuller left a comment

Choose a reason for hiding this comment

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

LGTM - although this is OK as is, it's a little easier to read with sensible item numbers.

@shihaohong shihaohong merged commit 96003f9 into flutter:master Mar 2, 2020
@shihaohong shihaohong deleted the locale-doc-fix branch March 2, 2020 17:49
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

d: api docs Issues with https://api.flutter.dev/ framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants