Uh oh!
There was an error while loading. Please reload this page.
Added Badge.count constructor - #115297
Merged
Merged
Conversation
darrenaustin
approved these changes
Nov 14, 2022
darrenaustin
left a comment
Contributor
There was a problem hiding this comment.
LGTM. Just a question about the test. Thx!
Contributor
There was a problem hiding this comment.
Suggested change
| // theme.textTtheme is updated when the MaterialApp is built. | |
| // theme.textTheme is updated when the MaterialApp is built. |
Contributor
There was a problem hiding this comment.
Couldn't this just be:
Suggested change
| latefinalThemeData theme; | |
| latefinalThemeData theme=ThemeData.light(useMaterial3:true); |
And then you wouldn't have to pull it from the context later in the build? You could use it for both the build method and the expect clauses. Or I am missing something with the Builder widget?
ContributorAuthor
There was a problem hiding this comment.
ThemeData.textTheme is updated lazily so the initial value of ThemeData.light(useMaterial3: true).textTheme.labelSmall isn't the final value.
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Nov 17, 2022
HansMuller pushed a commit
that referenced
this pull request
Nov 17, 2022
This reverts commit e9622ce.
HansMuller added a commit
that referenced
this pull request
Nov 17, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Nov 17, 2022
auto-submitBot
pushed a commit
to flutter/packages
that referenced
this pull request
Nov 17, 2022
* d359af2 a062a2063 [macOS] Merge FlutterMetalCompositor, superclass (flutter-team-archive/engine#37648) (flutter/flutter#115474) * cacf1d2 Roll Flutter Engine from a062a20638d1 to 97974ebb5ff8 (3 revisions) (flutter/flutter#115478) * 490fe28 Updated instructions for adding new localized messages. (flutter/flutter#115390) * 84357cb re-enable firebase release smoke test since infra outage is over (flutter/flutter#115402) * f205471 Use concrete factory constructor for DomXMLHttpRequest (flutter/flutter#115094) * 55927d8 3c83b275a [Impeller] Reland: Refactor color source resolution to use explicit factory types (flutter-team-archive/engine#37677) (flutter/flutter#115482) * 829e44d Fixed label alignment (flutter/flutter#115409) * b579109 Roll Flutter Engine from 3c83b275a511 to 35ecb2bf4139 (2 revisions) (flutter/flutter#115495) * d01874d [framework] re-rasterize page transition when layout size changes (flutter/flutter#115371) * 68740a2 fee463552 Roll Fuchsia Linux SDK from T7_W8eoMNuMdG-5R9... to haqxMhaSf7-4ZeVL8... (flutter-team-archive/engine#37684) (flutter/flutter#115498) * 26a990f Roll Flutter Engine from fee463552a93 to c5e6a38dd0bc (2 revisions) (flutter/flutter#115504) * f255390 Remove redundant arguments passed to redirecting factory constructors (flutter/flutter#115497) * 20139c6 4f3f45617 Move windows engine_v2 to prod. (flutter-team-archive/engine#37680) (flutter/flutter#115508) * 2ad4dc5 dcdb96e9d Roll Skia from 1dfecd7cd478 to 7f385b0ae434 (9 revisions) (flutter-team-archive/engine#37693) (flutter/flutter#115512) * 9c54635 Bump codecov/codecov-action from 3.1.0 to 3.1.1 (flutter/flutter#113126) * fafa11d Roll Flutter Engine from dcdb96e9db6b to cb403c1feaee (2 revisions) (flutter/flutter#115516) * 59f389d Remove unneeded comparison to `double.nan` (flutter/flutter#115470) * c071950 d422023f3 Migrate const_finder tests to be null safe (flutter-team-archive/engine#37683) (flutter/flutter#115524) * bf6d0f3 5bbf070c3 Roll Skia from 7f385b0ae434 to 8589c8ac6ee6 (1 revision) (flutter-team-archive/engine#37702) (flutter/flutter#115531) * 83d205e 7156b87c7 Roll Skia from 8589c8ac6ee6 to 68784d303cd6 (4 revisions) (flutter-team-archive/engine#37705) (flutter/flutter#115538) * 7e8568b 83c2482d7 [Impeller] Add Impeller scene (flutter-team-archive/engine#37694) (flutter/flutter#115541) * 07de108 ad3038c25 Roll Skia from 68784d303cd6 to 95ac3b53be37 (1 revision) (flutter-team-archive/engine#37706) (flutter/flutter#115548) * 1b23ad6 Roll Flutter Engine from ad3038c25cfa to 832aae2f850a (2 revisions) (flutter/flutter#115552) * e8cbd44 Added Badge.isLabelVisible flag (flutter/flutter#115292) * e9622ce Added Badge.count constructor (flutter/flutter#115297) * 537f89c Revert "Added Badge.count constructor (#115297)" (flutter/flutter#115564)
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 17, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
engine-flutter-autoroll added a commit
to engine-flutter-autoroll/plugins
that referenced
this pull request
Nov 18, 2022
shogohida pushed a commit
to shogohida/flutter
that referenced
this pull request
Dec 7, 2022
shogohida pushed a commit
to shogohida/flutter
that referenced
this pull request
Dec 7, 2022
This reverts commit e9622ce. Landing before presubmits have completed to unblock the build.
gspencergoog pushed a commit
to gspencergoog/flutter
that referenced
this pull request
Jan 19, 2023
gspencergoog pushed a commit
to gspencergoog/flutter
that referenced
this pull request
Jan 19, 2023
This reverts commit e9622ce. Landing before presubmits have completed to unblock the build.
creatorpiyush pushed a commit
to creatorpiyush/packages
that referenced
this pull request
Jun 10, 2026
* d359af2 a062a2063 [macOS] Merge FlutterMetalCompositor, superclass (flutter-team-archive/engine#37648) (flutter/flutter#115474) * cacf1d2 Roll Flutter Engine from a062a20638d1 to 97974ebb5ff8 (3 revisions) (flutter/flutter#115478) * 490fe28 Updated instructions for adding new localized messages. (flutter/flutter#115390) * 84357cb re-enable firebase release smoke test since infra outage is over (flutter/flutter#115402) * f205471 Use concrete factory constructor for DomXMLHttpRequest (flutter/flutter#115094) * 55927d8 3c83b275a [Impeller] Reland: Refactor color source resolution to use explicit factory types (flutter-team-archive/engine#37677) (flutter/flutter#115482) * 829e44d Fixed label alignment (flutter/flutter#115409) * b579109 Roll Flutter Engine from 3c83b275a511 to 35ecb2bf4139 (2 revisions) (flutter/flutter#115495) * d01874d [framework] re-rasterize page transition when layout size changes (flutter/flutter#115371) * 68740a2 fee463552 Roll Fuchsia Linux SDK from T7_W8eoMNuMdG-5R9... to haqxMhaSf7-4ZeVL8... (flutter-team-archive/engine#37684) (flutter/flutter#115498) * 26a990f Roll Flutter Engine from fee463552a93 to c5e6a38dd0bc (2 revisions) (flutter/flutter#115504) * f255390 Remove redundant arguments passed to redirecting factory constructors (flutter/flutter#115497) * 20139c6 4f3f45617 Move windows engine_v2 to prod. (flutter-team-archive/engine#37680) (flutter/flutter#115508) * 2ad4dc5 dcdb96e9d Roll Skia from 1dfecd7cd478 to 7f385b0ae434 (9 revisions) (flutter-team-archive/engine#37693) (flutter/flutter#115512) * 9c54635 Bump codecov/codecov-action from 3.1.0 to 3.1.1 (flutter/flutter#113126) * fafa11d Roll Flutter Engine from dcdb96e9db6b to cb403c1feaee (2 revisions) (flutter/flutter#115516) * 59f389d Remove unneeded comparison to `double.nan` (flutter/flutter#115470) * c071950 d422023f3 Migrate const_finder tests to be null safe (flutter-team-archive/engine#37683) (flutter/flutter#115524) * bf6d0f3 5bbf070c3 Roll Skia from 7f385b0ae434 to 8589c8ac6ee6 (1 revision) (flutter-team-archive/engine#37702) (flutter/flutter#115531) * 83d205e 7156b87c7 Roll Skia from 8589c8ac6ee6 to 68784d303cd6 (4 revisions) (flutter-team-archive/engine#37705) (flutter/flutter#115538) * 7e8568b 83c2482d7 [Impeller] Add Impeller scene (flutter-team-archive/engine#37694) (flutter/flutter#115541) * 07de108 ad3038c25 Roll Skia from 68784d303cd6 to 95ac3b53be37 (1 revision) (flutter-team-archive/engine#37706) (flutter/flutter#115548) * 1b23ad6 Roll Flutter Engine from ad3038c25cfa to 832aae2f850a (2 revisions) (flutter/flutter#115552) * e8cbd44 Added Badge.isLabelVisible flag (flutter/flutter#115292) * e9622ce Added Badge.count constructor (flutter/flutter#115297) * 537f89c Revert "Added Badge.count constructor (#115297)" (flutter/flutter#115564)
bisor0627 pushed a commit
to bisor0627/packages
that referenced
this pull request
Jun 19, 2026
* d359af2 a062a2063 [macOS] Merge FlutterMetalCompositor, superclass (flutter-team-archive/engine#37648) (flutter/flutter#115474) * cacf1d2 Roll Flutter Engine from a062a20638d1 to 97974ebb5ff8 (3 revisions) (flutter/flutter#115478) * 490fe28 Updated instructions for adding new localized messages. (flutter/flutter#115390) * 84357cb re-enable firebase release smoke test since infra outage is over (flutter/flutter#115402) * f205471 Use concrete factory constructor for DomXMLHttpRequest (flutter/flutter#115094) * 55927d8 3c83b275a [Impeller] Reland: Refactor color source resolution to use explicit factory types (flutter-team-archive/engine#37677) (flutter/flutter#115482) * 829e44d Fixed label alignment (flutter/flutter#115409) * b579109 Roll Flutter Engine from 3c83b275a511 to 35ecb2bf4139 (2 revisions) (flutter/flutter#115495) * d01874d [framework] re-rasterize page transition when layout size changes (flutter/flutter#115371) * 68740a2 fee463552 Roll Fuchsia Linux SDK from T7_W8eoMNuMdG-5R9... to haqxMhaSf7-4ZeVL8... (flutter-team-archive/engine#37684) (flutter/flutter#115498) * 26a990f Roll Flutter Engine from fee463552a93 to c5e6a38dd0bc (2 revisions) (flutter/flutter#115504) * f255390 Remove redundant arguments passed to redirecting factory constructors (flutter/flutter#115497) * 20139c6 4f3f45617 Move windows engine_v2 to prod. (flutter-team-archive/engine#37680) (flutter/flutter#115508) * 2ad4dc5 dcdb96e9d Roll Skia from 1dfecd7cd478 to 7f385b0ae434 (9 revisions) (flutter-team-archive/engine#37693) (flutter/flutter#115512) * 9c54635 Bump codecov/codecov-action from 3.1.0 to 3.1.1 (flutter/flutter#113126) * fafa11d Roll Flutter Engine from dcdb96e9db6b to cb403c1feaee (2 revisions) (flutter/flutter#115516) * 59f389d Remove unneeded comparison to `double.nan` (flutter/flutter#115470) * c071950 d422023f3 Migrate const_finder tests to be null safe (flutter-team-archive/engine#37683) (flutter/flutter#115524) * bf6d0f3 5bbf070c3 Roll Skia from 7f385b0ae434 to 8589c8ac6ee6 (1 revision) (flutter-team-archive/engine#37702) (flutter/flutter#115531) * 83d205e 7156b87c7 Roll Skia from 8589c8ac6ee6 to 68784d303cd6 (4 revisions) (flutter-team-archive/engine#37705) (flutter/flutter#115538) * 7e8568b 83c2482d7 [Impeller] Add Impeller scene (flutter-team-archive/engine#37694) (flutter/flutter#115541) * 07de108 ad3038c25 Roll Skia from 68784d303cd6 to 95ac3b53be37 (1 revision) (flutter-team-archive/engine#37706) (flutter/flutter#115548) * 1b23ad6 Roll Flutter Engine from ad3038c25cfa to 832aae2f850a (2 revisions) (flutter/flutter#115552) * e8cbd44 Added Badge.isLabelVisible flag (flutter/flutter#115292) * e9622ce Added Badge.count constructor (flutter/flutter#115297) * 537f89c Revert "Added Badge.count constructor (#115297)" (flutter/flutter#115564)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added a named
Badge.countconstructor as a convenience for the common case of creating a Badge whose label is an integer with 1-3 digits. Count values greater than 999 are displayed as '999+'.Per a suggest from @darrenaustin - #114560 (comment).