Skip to content

[material_ui] Update out-of-date typography documentation - #13372

Merged
auto-submit[bot] merged 2 commits into
flutter:mainfrom
elliette:typography-documentation
Oct 27, 2026
Merged

auto-submit[bot] merged 2 commits into
flutter:mainfrom
elliette:typography-documentation

Conversation

@elliette

@elliette elliette commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Updates out-of-date documentation for typography, which is missing reference to the 2021 theme (the current default).

Fixes flutter/flutter#195013

Pre-Review Checklist

Text exemption: Documentation changes only.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

Footnotes

  1. Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩2

@flutter-dashboard

Copy link
Copy Markdown

It looks like this pull request may not have tests. Please make sure to add tests or get an explicit test exemption before merging.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. If both you and the PR author believe this PR qualifies for a test exemption, escalate to the tech lead for the changed area to review and apply the test-exempt label. All reviewers should feel empowered to ask for tests before escalating.

@elliette elliette changed the title [ [material_ui] Update out-of-date typography documentation Oct 21, 2026
@github-actions github-actions Bot added p: material_ui triage-design Should be looked at in design triage labels Oct 21, 2026

@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 the Typography documentation in packages/material_ui/lib/src/typography.dart to describe the 2021 (Material 3) geometry themes and the current ThemeData.typography defaults. It also adds a pending changelog entry detailing these documentation updates. There are no review comments, and I have no feedback to provide.

@elliette
elliette requested a review from QuncCccccc October 22, 2026 16:23

@QuncCccccc QuncCccccc 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. Thank you for the correction!

@elliette elliette added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 23, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Oct 23, 2026
@auto-submit

auto-submit Bot commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/packages/13372, because - This pull request has the missing-tests label and does not have the test-exempt label. Please add tests or obtain a test exemption from a tech lead before re-applying the autosubmit label.

@QuncCccccc

Copy link
Copy Markdown
Contributor

The missing test label is added because we have changes in pending changelog. We don't need to request test exempt for this kind of changes. flutter/cocoon#5167 will fix it (the original issue is flutter/flutter#191744).

@elliette elliette added test-exempt Exempt from tests. Overrides `missing-tests` label autosubmit Merge PR when tree becomes green via auto submit App labels Oct 23, 2026
@auto-submit
auto-submit Bot merged commit d96c511 into flutter:main Oct 27, 2026
12 checks passed
GhagSagar23 pushed a commit to Kryonex-Labs/flutter that referenced this pull request Oct 27, 2026
flutter/packages@23a05b6...a8549ae

2026-10-27 [email protected] [material_ui]
Deprecate the black/white parameters in `Typography.material2021`
(flutter/packages#13443)
2026-10-27 [email protected] [shared_preferences] Defer
SharedPreferences fetch in `LegacySharedPreferencesPlugin` until first
use on Android (flutter/packages#13431)
2026-10-27 [email protected] [material_ui] Fix MenuAnchor
non-Unicode shortcut labels on JS web (flutter/packages#13370)
2026-10-27 [email protected] [material_ui]
Add Checkbox.markInsets (flutter/packages#12690)
2026-10-27 [email protected]
[vector_graphics_compiler] Handle coordinate lists on text and tspan
(flutter/packages#13287)
2026-10-27 [email protected]
[cupertino_ui] Unskip web tests that pass on CanvasKit and Skwasm:
nav_bar_transition_test, refresh_test, route_test,
text_field_cursor_test (flutter/packages#13439)
2026-10-27 [email protected] [material_ui] Fix
`TextTheme` and `Typography` `debugFillProperties` defaults
(flutter/packages#13374)
2026-10-27 [email protected] [material_ui]
Update out-of-date typography documentation (flutter/packages#13372)
2026-10-27 [email protected] [shared_preferences_foundation] Fix web
compilation of apps that import the package (flutter/packages#13447)
2026-10-26 [email protected] Fix AI guidelines link in PR template
(flutter/packages#13411)

If this roll has caused a breakage, revert this CL and set the roller
to dry run mode using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
suryaKommana2662 pushed a commit to suryaKommana2662/flutter that referenced this pull request Oct 29, 2026
flutter/packages@23a05b6...a8549ae

2026-10-27 [email protected] [material_ui]
Deprecate the black/white parameters in `Typography.material2021`
(flutter/packages#13443)
2026-10-27 [email protected] [shared_preferences] Defer
SharedPreferences fetch in `LegacySharedPreferencesPlugin` until first
use on Android (flutter/packages#13431)
2026-10-27 [email protected] [material_ui] Fix MenuAnchor
non-Unicode shortcut labels on JS web (flutter/packages#13370)
2026-10-27 [email protected] [material_ui]
Add Checkbox.markInsets (flutter/packages#12690)
2026-10-27 [email protected]
[vector_graphics_compiler] Handle coordinate lists on text and tspan
(flutter/packages#13287)
2026-10-27 [email protected]
[cupertino_ui] Unskip web tests that pass on CanvasKit and Skwasm:
nav_bar_transition_test, refresh_test, route_test,
text_field_cursor_test (flutter/packages#13439)
2026-10-27 [email protected] [material_ui] Fix
`TextTheme` and `Typography` `debugFillProperties` defaults
(flutter/packages#13374)
2026-10-27 [email protected] [material_ui]
Update out-of-date typography documentation (flutter/packages#13372)
2026-10-27 [email protected] [shared_preferences_foundation] Fix web
compilation of apps that import the package (flutter/packages#13447)
2026-10-26 [email protected] Fix AI guidelines link in PR template
(flutter/packages#13411)

If this roll has caused a breakage, revert this CL and set the roller
to dry run mode using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD missing-tests p: material_ui test-exempt Exempt from tests. Overrides `missing-tests` label triage-design Should be looked at in design triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[material_ui] Out-of-date documentation in typography.dart

2 participants