Feat: Add brightnessOf method for theme #163733

rkishan516 · 2025-02-20T15:40:06Z

Feat: Add brightnessOf method for theme
fixes: #163393

Pre-launch Checklist

I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
I read the [Tree Hygiene] wiki page, which explains my responsibilities.
I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
I signed the [CLA].
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is [test-exempt].
I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
All existing and new tests are passing.

rkishan516 · 2025-02-20T15:41:25Z

This PR doesn't include implementation of InheritedModel for Theme.

navaronbracke · 2025-02-21T12:07:18Z

packages/flutter/lib/src/material/theme.dart

+    return inheritedTheme?.theme.data.brightness ?? MediaQuery.platformBrightnessOf(context);
+  }
+
+  /// Retrieves the [Brightness] to use for descendant Cupertino widgets, based


Same here

Suggested change

/// Retrieves the [Brightness] to use for descendant Cupertino widgets, based

/// Retrieves the [Brightness] to use for descendant Material widgets, based

/// on the value of [ThemeData.brightness] in the given [context].

///

/// If no [InheritedTheme] can be found in the given [context], it will fall

/// back to [MediaQueryData.platformBrightness].

///

/// Returns null if no valid [InheritedTheme] or [MediaQuery] widgets exist in

/// the ancestry tree.

///

/// See also:

///

/// * [ThemeData.brightness], the property that takes precedence over

/// [MediaQueryData.platformBrightness] for descendant Material widgets.

/// * [brightnessOf], which throws if no valid [InheritedTheme] or

/// [MediaQuery] exists, instead of returning null.

navaronbracke · 2025-02-21T12:13:24Z

packages/flutter/test/material/theme_test.dart

@@ -44,7 +44,7 @@ void main() {
    await tester.tap(find.byKey(popupMenuButtonKey));
    await tester.pump(const Duration(seconds: 1));

-    expect(Theme.of(tester.element(find.text('menuItem'))).brightness, equals(Brightness.dark));
+    expect(Theme.brightnessOf(tester.element(find.text('menuItem'))), equals(Brightness.dark));


I would propose to not do this change in this PR, we can probably do all the ThemeData.brightness changes in a follow up PR, after this one lands. (which probably makes it a bit easier to review)

Instead, I would test the behavior of brightnessOf / maybeBrightnessOf in separate tests:

brightnessOf returns the correct brightness if only a MediaQuery is provided

brightnessOf returns the correct brightness if both a MediaQuery & ThemeData are provided

brightnessOf throws if neither a MediaQuery or ThemeData is provided

maybeBrightnessOf returns the correct brightness if only a MediaQuery is provided

maybeBrightnessOf returns the correct brightness if both a MediaQuery & ThemeData are provided

maybeBrightnessOf returns null if neither a MediaQuery or ThemeData is provided

I will file a follow up issue to do the former sparately.

Sure, let me create another test which just verify that Theme.brightnessOf works in all scenario

@navaronbracke I don't think scenario 3 and 6 can be produced, because in testing media query always return platform brightness.

For these tests I would just pump a widget with the WidgetTester:

Only pumps a MediaQuery widget with a Builder as child (to get the context for the brightness test)

Pumps both a MediaQuery & a Theme widget, with a Builder as child

Pumps only a Builder widget and never pumps a MediaQuery or Theme widget

Cases 4-6 are variations, but use the other method.

You should be able to give the MediaQuery a custom MediaQueryData, to force the brightness?

Yes, MediaQuery takes data as input.

navaronbracke · 2025-02-21T13:08:00Z

packages/flutter/test/material/theme_test.dart

+      await tester.pumpWidget(
+        MediaQuery(
+          data: const MediaQueryData(platformBrightness: Brightness.dark),
+          child: Container(),


I would use a SizedBox, instead of a Container to make the tests a little lighter.

navaronbracke · 2025-02-21T13:10:03Z

packages/flutter/test/material/theme_test.dart

+      await tester.pumpWidget(
+        Theme(
+          data: ThemeData(brightness: Brightness.light),
+          child: MediaQuery(


Should the MediaQuery be above the Theme, for consistency with the real widget tree in MaterialApp?

navaronbracke · 2025-02-21T13:13:04Z

packages/flutter/lib/src/material/theme.dart

+  ///
+  /// See also:
+  ///
+  /// * [ThemeData.brightness], the property takes precedence over


Suggested change

/// * [ThemeData.brightness], the property takes precedence over

/// * [ThemeData.brightness], the property that takes precedence over

navaronbracke · 2025-02-21T13:13:57Z

packages/flutter/lib/src/material/theme.dart

+  ///
+  /// * [maybeBrightnessOf], which returns null if no valid [InheritedTheme] or
+  ///   [MediaQuery] exists, instead of throwing.
+  /// * [ThemeData.brightness], the property takes precedence over


Suggested change

/// * [ThemeData.brightness], the property takes precedence over

/// * [ThemeData.brightness], the property that takes precedence over

navaronbracke · 2025-02-21T13:15:29Z

packages/flutter/lib/src/material/theme.dart

+  /// * [maybeBrightnessOf], which returns null if no valid [InheritedTheme] or
+  ///   [MediaQuery] exists, instead of throwing.
+  /// * [ThemeData.brightness], the property takes precedence over
+  ///   [MediaQueryData.platformBrightness] for descendant widgets.


I would keep the mention of Material, since Theme is a Material thing and not under the widgets library.

Suggested change

/// [MediaQueryData.platformBrightness] for descendant widgets.

/// [MediaQueryData.platformBrightness] for descendant Material widgets.

navaronbracke · 2025-02-21T13:16:22Z

packages/flutter/lib/src/material/theme.dart

@@ -161,6 +161,48 @@ class Theme extends StatelessWidget {
        .resolveFrom(context);
  }

+  /// Retrieves the [Brightness] to use for descendant widgets, based


I would keep the mention of Material, since Theme is a Material thing and not under the widgets library.

Suggested change

/// Retrieves the [Brightness] to use for descendant widgets, based

/// Retrieves the [Brightness] to use for descendant Material widgets, based

navaronbracke · 2025-02-21T13:17:09Z

packages/flutter/lib/src/material/theme.dart

+    return inheritedTheme?.theme.data.brightness ?? MediaQuery.platformBrightnessOf(context);
+  }
+
+  /// Retrieves the [Brightness] to use for descendant widgets, based


I would keep the mention of Material, since Theme is a Material thing and not under the widgets library.

Suggested change

/// Retrieves the [Brightness] to use for descendant widgets, based

/// Retrieves the [Brightness] to use for descendant Material widgets, based

navaronbracke · 2025-02-21T13:17:35Z

packages/flutter/lib/src/material/theme.dart

+  /// See also:
+  ///
+  /// * [ThemeData.brightness], the property takes precedence over
+  ///   [MediaQueryData.platformBrightness] for descendant widgets.


I would keep the mention of Material, since Theme is a Material thing and not under the widgets library.

Suggested change

/// [MediaQueryData.platformBrightness] for descendant widgets.

/// [MediaQueryData.platformBrightness] for descendant Material widgets.

navaronbracke

Minor docs correction and a suggestion for some missing tests.

navaronbracke · 2025-02-22T10:53:36Z

packages/flutter/lib/src/material/theme.dart

+  /// If no [InheritedTheme] can be found in the given [context], or its `brightness`
+  /// is null, it will fall back to [MediaQueryData.platformBrightness].
+  ///
+  /// Throws an exception if no valid [InheritedTheme] or [MediaQuery] widgets


Suggested change

/// Throws an exception if no valid [InheritedTheme] or [MediaQuery] widgets

I had a look at the implementation of https://api.flutter.dev/flutter/widgets/MediaQuery/platformBrightnessOf.html

and it returns Brightness.light instead of throwing if there is no MediaQuery.
So you can remove the phrase that says it throws.

packages/flutter/test/material/theme_test.dart

navaronbracke

Some minor docs adjustments, to match the actual behavior of the method that is used, along with a clarification for one test.

navaronbracke · 2025-02-22T12:44:50Z

packages/flutter/lib/src/material/theme.dart

+  /// See also:
+  ///
+  /// * [maybeBrightnessOf], which returns null if no valid [InheritedTheme] or
+  ///   [MediaQuery] exists, instead of throwing.


Since platformBrightnessOf does not throw, but returns a default, I think we can tweak the wording here.

Suggested change

/// [MediaQuery] exists, instead of throwing.

/// [MediaQuery] exists.

navaronbracke · 2025-02-22T12:46:32Z

packages/flutter/lib/src/material/theme.dart

+  /// If no [InheritedTheme] or [MediaQuery] can be found in the given [context], it will
+  /// return null.
+  ///
+  /// Returns null if no valid [InheritedTheme] or [MediaQuery] widgets exist in


Can we remove this phrase? Currently it says returns null twice

navaronbracke · 2025-02-22T12:47:30Z

packages/flutter/lib/src/material/theme.dart

+  ///
+  /// * [ThemeData.brightness], the property that takes precedence over
+  ///   [MediaQueryData.platformBrightness] for descendant Material widgets.
+  /// * [brightnessOf], which throws if no valid [InheritedTheme] or


Same tweak here, since that method does not throw.

Suggested change

/// * [brightnessOf], which throws if no valid [InheritedTheme] or

/// * [brightnessOf], which returns a default value if no valid [InheritedTheme] or

navaronbracke · 2025-02-22T12:50:19Z

packages/flutter/test/material/theme_test.dart

+    testWidgets('returns Brightness.light when no theme or media query is present', (
+      WidgetTester tester,
+    ) async {
+      await tester.pumpWidget(const SizedBox());


We need to prevent the View from adding the MediaQuery here, otherwise there is an implicit MediaQuery

Suggested change

await tester.pumpWidget(const SizedBox());

// Prevent the implicitly added View from adding a MediaQuery

await tester.pumpWidget(

RawView(view: FakeFlutterView(tester.view, viewId: 77), child: const SizedBox()),

wrapWithView: false,

);

navaronbracke · 2025-02-22T15:18:28Z

@rkishan516 While to me this PR looks good now, I will have to defer to people on the framework team for a review / explicit approval. So you will have to be patient a little :)

See https://github.com/flutter/flutter/blob/master/docs/contributing/Tree-hygiene.md#when for more info on the review process.

rkishan516 · 2025-02-22T15:55:07Z

@rkishan516 While to me this PR looks good now, I will have to defer to people on the framework team for a review / explicit approval. So you will have to be patient a little :)

See https://github.com/flutter/flutter/blob/master/docs/contributing/Tree-hygiene.md#when for more info on the review process.

Sure and Thanks for helping @navaronbracke.

QuncCccccc

Can we use Theme.of(context).brightness to detect the brightness?

navaronbracke · 2025-03-14T19:46:35Z

That is how you would do this today. However, this PR aims to make it explicit, using a named method, and is a step towards eventually making Theme an InheritedModel. The latter is the main reasoning behind this change.

dkwingsmt · 2025-03-14T20:05:26Z

@navaronbracke I'm not familiar with this work, but is there an issue tracking our progress (and discussions, if any) towards making Theme an InheritedModel?

navaronbracke · 2025-03-14T20:10:32Z

@nate-thegrate made a design doc that talks about this a little in https://docs.google.com/document/d/1MfU6gkgIgXCxa5E25sPYuAP5Bmgehyl3Hk7yDH24zl0/edit?tab=t.0

dkwingsmt · 2025-03-14T20:25:54Z

I've asked @MitchellGoodwin and it seems the conversation didn't reached a point were a decision was made. Although I agree with the motivation of the work, I think it'd be better not to move forward until we have agreed on the exact design and plan.

If you would like to drive this project, you can start by writing a more detailed design doc with thorough planning and introduction, and we can seek comments or even host a Dash forum for discussion.

navaronbracke · 2025-03-14T21:30:17Z

Outside of InheritedModel, there is value in adding a brightnessOf method to Material for the sake of API parity since Cupertino has a brightnessOf method. Material and Cupertino handle their brightness differently though, in such a way were I don't think this method would decrease rebuilds the way it does for Cupertino theme.

Yes, feature parity between the two was also something I mentioned in the OP of #163393

rkishan516 · 2025-03-14T22:37:06Z

Outside of InheritedModel, there is value in adding a brightnessOf method to Material for the sake of API parity since Cupertino has a brightnessOf method. Material and Cupertino handle their brightness differently though, in such a way were I don't think this method would decrease rebuilds the way it does for Cupertino theme.

If we are able to push Theme as InheritedModel, then later part will not be an issue. If we decide to go with migrating Theme to InheritedModel, I won't mind doing that change.

rkishan516 · 2025-03-26T07:21:11Z

@dkwingsmt or @MitchellGoodwin Any update on this ?

dkwingsmt

API parity sounds a convincing reason. LGTM. Thank you!

MitchellGoodwin

LGTM!

…3733)

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/flutter@05b5e79...a0b1b32 2025-04-01 [email protected] Roll Dart SDK from b4d374ec59ec to 4e1f02bc704f (2 revisions) (flutter/flutter#166342) 2025-04-01 [email protected] Trim any text before osascript JSON response (flutter/flutter#166296) 2025-04-01 [email protected] [Gen-l10n] Add `Message.resourceId` and `locale` to all `L10nException` error messages (flutter/flutter#163654) 2025-04-01 [email protected] Add `--ignore-timeouts` flag for `flutter test` command (flutter/flutter#164437) 2025-04-01 [email protected] Update TESTOWNERS username (flutter/flutter#166191) 2025-04-01 [email protected] Roll Skia from 4b07443e6071 to 52cbb917fffd (4 revisions) (flutter/flutter#166329) 2025-04-01 [email protected] Roll Dart SDK from 6b07a09cbd2d to b4d374ec59ec (2 revisions) (flutter/flutter#166321) 2025-04-01 [email protected] [tool] Improve using project files in build targets (flutter/flutter#166211) 2025-04-01 [email protected] Rename FlRenderer to FlCompositorOpenGL (flutter/flutter#166037) 2025-04-01 [email protected] [engine, web_ui] Fix instances of library_private_types_in_public_api (flutter/flutter#166156) 2025-04-01 [email protected] Roll Dart SDK from 509faa921c95 to 6b07a09cbd2d (1 revision) (flutter/flutter#166301) 2025-04-01 [email protected] [Impeller] small cpu perf for text contents. (flutter/flutter#166199) 2025-04-01 [email protected] [android_engine_test] disable old HC mode tests. (flutter/flutter#166293) 2025-04-01 [email protected] [impeller] fixes diagonal antialiased lines (flutter/flutter#166298) 2025-04-01 [email protected] Roll Skia from 5f262bd2cbb4 to 4b07443e6071 (10 revisions) (flutter/flutter#166299) 2025-03-31 [email protected] [Impeller] Directly tessellate conics to linear path segments (flutter/flutter#166165) 2025-03-31 [email protected] [tool] Don't write the .flutter-plugins-dependencies file if it is unchanged (flutter/flutter#166164) 2025-03-31 [email protected] Move `.cxx` directory out of `android/app` (flutter/flutter#166277) 2025-03-31 [email protected] Fix typo in carousel.dart (flutter/flutter#164727) 2025-03-31 [email protected] Roll Dart SDK from c5fa06710bb6 to 509faa921c95 (1 revision) (flutter/flutter#166283) 2025-03-31 [email protected] Public nodes needing paint or layout (flutter/flutter#166148) 2025-03-31 [email protected] [Gen-l10n] Infer placeholder types on both templates and localizations (flutter/flutter#163690) 2025-03-31 [email protected] [Engine][iOS] Cancel animation when recieved `UIKeyboardWillHideNotification` with duration 0.0 (flutter/flutter#164884) 2025-03-31 [email protected] [fuchsia] Remove explicit LogSink and InspectSink routing and use dictionaries instead (flutter/flutter#162780) 2025-03-31 [email protected] Updated to latest AVD to Support Android 16 (API 36) (flutter/flutter#165926) 2025-03-31 [email protected] Feat: Add brightnessOf method for theme (flutter/flutter#163733) 2025-03-31 [email protected] Marks Linux_mokey new_gallery__crane_perf to be flaky (flutter/flutter#165964) 2025-03-31 [email protected] [ Tool ] Correctly select entrypoint target for web build from positional argument list (flutter/flutter#166260) 2025-03-31 [email protected] [Impeller] remove validation warning ignores. (flutter/flutter#166205) 2025-03-31 [email protected] [Impeller] handle shader ordering bug on macOS. (flutter/flutter#165937) 2025-03-31 [email protected] Fix CODEOWNERS for the iOS review team (flutter/flutter#166178) 2025-03-31 [email protected] Remove `<meta content="IE=Edge" http-equiv="X-UA-Compatible">` (flutter/flutter#166252) 2025-03-31 [email protected] Roll Dart SDK from b9c35e05feb5 to c5fa06710bb6 (1 revision) (flutter/flutter#166251) 2025-03-31 [email protected] Roll Skia from 418c68ea5ccb to 5f262bd2cbb4 (2 revisions) (flutter/flutter#166244) 2025-03-31 [email protected] Roll Skia from b6a3bbd1d153 to 418c68ea5ccb (1 revision) (flutter/flutter#166236) 2025-03-31 [email protected] [Impeller] fix min filter for GL external textures. (flutter/flutter#166224) 2025-03-31 [email protected] Roll Skia from 10f4cf9a817d to b6a3bbd1d153 (13 revisions) (flutter/flutter#166231) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose ...

…r#8985) Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/flutter@05b5e79...a0b1b32 2025-04-01 [email protected] Roll Dart SDK from b4d374ec59ec to 4e1f02bc704f (2 revisions) (flutter/flutter#166342) 2025-04-01 [email protected] Trim any text before osascript JSON response (flutter/flutter#166296) 2025-04-01 [email protected] [Gen-l10n] Add `Message.resourceId` and `locale` to all `L10nException` error messages (flutter/flutter#163654) 2025-04-01 [email protected] Add `--ignore-timeouts` flag for `flutter test` command (flutter/flutter#164437) 2025-04-01 [email protected] Update TESTOWNERS username (flutter/flutter#166191) 2025-04-01 [email protected] Roll Skia from 4b07443e6071 to 52cbb917fffd (4 revisions) (flutter/flutter#166329) 2025-04-01 [email protected] Roll Dart SDK from 6b07a09cbd2d to b4d374ec59ec (2 revisions) (flutter/flutter#166321) 2025-04-01 [email protected] [tool] Improve using project files in build targets (flutter/flutter#166211) 2025-04-01 [email protected] Rename FlRenderer to FlCompositorOpenGL (flutter/flutter#166037) 2025-04-01 [email protected] [engine, web_ui] Fix instances of library_private_types_in_public_api (flutter/flutter#166156) 2025-04-01 [email protected] Roll Dart SDK from 509faa921c95 to 6b07a09cbd2d (1 revision) (flutter/flutter#166301) 2025-04-01 [email protected] [Impeller] small cpu perf for text contents. (flutter/flutter#166199) 2025-04-01 [email protected] [android_engine_test] disable old HC mode tests. (flutter/flutter#166293) 2025-04-01 [email protected] [impeller] fixes diagonal antialiased lines (flutter/flutter#166298) 2025-04-01 [email protected] Roll Skia from 5f262bd2cbb4 to 4b07443e6071 (10 revisions) (flutter/flutter#166299) 2025-03-31 [email protected] [Impeller] Directly tessellate conics to linear path segments (flutter/flutter#166165) 2025-03-31 [email protected] [tool] Don't write the .flutter-plugins-dependencies file if it is unchanged (flutter/flutter#166164) 2025-03-31 [email protected] Move `.cxx` directory out of `android/app` (flutter/flutter#166277) 2025-03-31 [email protected] Fix typo in carousel.dart (flutter/flutter#164727) 2025-03-31 [email protected] Roll Dart SDK from c5fa06710bb6 to 509faa921c95 (1 revision) (flutter/flutter#166283) 2025-03-31 [email protected] Public nodes needing paint or layout (flutter/flutter#166148) 2025-03-31 [email protected] [Gen-l10n] Infer placeholder types on both templates and localizations (flutter/flutter#163690) 2025-03-31 [email protected] [Engine][iOS] Cancel animation when recieved `UIKeyboardWillHideNotification` with duration 0.0 (flutter/flutter#164884) 2025-03-31 [email protected] [fuchsia] Remove explicit LogSink and InspectSink routing and use dictionaries instead (flutter/flutter#162780) 2025-03-31 [email protected] Updated to latest AVD to Support Android 16 (API 36) (flutter/flutter#165926) 2025-03-31 [email protected] Feat: Add brightnessOf method for theme (flutter/flutter#163733) 2025-03-31 [email protected] Marks Linux_mokey new_gallery__crane_perf to be flaky (flutter/flutter#165964) 2025-03-31 [email protected] [ Tool ] Correctly select entrypoint target for web build from positional argument list (flutter/flutter#166260) 2025-03-31 [email protected] [Impeller] remove validation warning ignores. (flutter/flutter#166205) 2025-03-31 [email protected] [Impeller] handle shader ordering bug on macOS. (flutter/flutter#165937) 2025-03-31 [email protected] Fix CODEOWNERS for the iOS review team (flutter/flutter#166178) 2025-03-31 [email protected] Remove `<meta content="IE=Edge" http-equiv="X-UA-Compatible">` (flutter/flutter#166252) 2025-03-31 [email protected] Roll Dart SDK from b9c35e05feb5 to c5fa06710bb6 (1 revision) (flutter/flutter#166251) 2025-03-31 [email protected] Roll Skia from 418c68ea5ccb to 5f262bd2cbb4 (2 revisions) (flutter/flutter#166244) 2025-03-31 [email protected] Roll Skia from b6a3bbd1d153 to 418c68ea5ccb (1 revision) (flutter/flutter#166236) 2025-03-31 [email protected] [Impeller] fix min filter for GL external textures. (flutter/flutter#166224) 2025-03-31 [email protected] Roll Skia from 10f4cf9a817d to b6a3bbd1d153 (13 revisions) (flutter/flutter#166231) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose ...

github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Feb 20, 2025

navaronbracke reviewed Feb 21, 2025

View reviewed changes

navaronbracke mentioned this pull request Feb 21, 2025

[Proposal] Migrate framework Material widgets to Theme.brightnessOf(context) #163837

Closed

rkishan516 force-pushed the brightness-of-theme branch from 4226de7 to c2e7da0 Compare February 21, 2025 12:44

rkishan516 requested a review from navaronbracke February 21, 2025 12:51

navaronbracke reviewed Feb 21, 2025

View reviewed changes

rkishan516 force-pushed the brightness-of-theme branch from c2e7da0 to f1770bd Compare February 21, 2025 14:52

rkishan516 requested a review from navaronbracke February 21, 2025 16:06

navaronbracke reviewed Feb 22, 2025

View reviewed changes

rkishan516 force-pushed the brightness-of-theme branch 3 times, most recently from 3cfa06b to b6c93e8 Compare February 22, 2025 11:54

rkishan516 requested a review from navaronbracke February 22, 2025 12:40

navaronbracke reviewed Feb 22, 2025

View reviewed changes

rkishan516 force-pushed the brightness-of-theme branch from b6c93e8 to 26f49a7 Compare February 22, 2025 13:25

rkishan516 requested a review from navaronbracke February 22, 2025 15:10

rkishan516 force-pushed the brightness-of-theme branch 2 times, most recently from 95dd5a5 to 227f414 Compare March 1, 2025 01:20

dkwingsmt requested review from dkwingsmt and QuncCccccc and removed request for navaronbracke March 12, 2025 18:50

QuncCccccc reviewed Mar 14, 2025

View reviewed changes

Feat: Add brightnessOf method for theme

1f075f8

rkishan516 force-pushed the brightness-of-theme branch from bf5e57e to 1f075f8 Compare March 15, 2025 02:18

rkishan516 mentioned this pull request Mar 17, 2025

Feat: Migrate Theme to InheritedModel #165289

Open

9 tasks

dkwingsmt approved these changes Mar 28, 2025

View reviewed changes

dkwingsmt requested a review from MitchellGoodwin March 28, 2025 23:16

MitchellGoodwin approved these changes Mar 31, 2025

View reviewed changes

MitchellGoodwin added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 31, 2025

auto-submit bot added this pull request to the merge queue Mar 31, 2025

Merged via the queue into flutter:master with commit 082a24c Mar 31, 2025
75 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 31, 2025

matanlurey mentioned this pull request Mar 31, 2025

Post-submit commits are not registering automatically on Cocoon #166272

Closed

rkishan516 mentioned this pull request Apr 1, 2025

Migrate to Theme.brightnessOf method #163950

Merged

9 tasks

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to 70cd54c3fd13 (259 revisions) flutter/packages#8973

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

910a8c5

…3733)

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to 0befaaafc952 (260 revisions) flutter/packages#8974

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

81b2eb3

…3733)

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to dbef1314b186 (261 revisions) flutter/packages#8976

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

9c9b907

…3733)

engine-flutter-autoroll mentioned this pull request Apr 1, 2025

Roll Flutter from 1d954f4e96bd to a0b1b3253416 (262 revisions) flutter/packages#8977

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

a593807

…3733)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 2, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

3922339

…3733)

engine-flutter-autoroll mentioned this pull request Apr 2, 2025

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/packages#8983

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 2, 2025

082a24cdb Feat: Add brightnessOf method for theme (flutter/flutter#16…

fc3c24e

…3733)

engine-flutter-autoroll mentioned this pull request Apr 2, 2025

Roll Flutter from 05b5e7910544 to a0b1b3253416 (37 revisions) flutter/packages#8985

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat: Add brightnessOf method for theme #163733

Feat: Add brightnessOf method for theme #163733

rkishan516 commented Feb 20, 2025

rkishan516 commented Feb 20, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

rkishan516 Feb 21, 2025 •

edited

Loading

rkishan516 Feb 21, 2025

navaronbracke Feb 21, 2025 •

edited

Loading

rkishan516 Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke Feb 21, 2025

navaronbracke left a comment

navaronbracke Feb 22, 2025

navaronbracke left a comment

navaronbracke Feb 22, 2025

navaronbracke Feb 22, 2025

navaronbracke Feb 22, 2025

navaronbracke Feb 22, 2025

navaronbracke commented Feb 22, 2025

rkishan516 commented Feb 22, 2025

QuncCccccc left a comment

navaronbracke commented Mar 14, 2025

dkwingsmt commented Mar 14, 2025

navaronbracke commented Mar 14, 2025

dkwingsmt commented Mar 14, 2025 •

edited

Loading

navaronbracke commented Mar 14, 2025

rkishan516 commented Mar 14, 2025

rkishan516 commented Mar 26, 2025

dkwingsmt left a comment

MitchellGoodwin left a comment

	/// * [ThemeData.brightness], the property takes precedence over
	/// * [ThemeData.brightness], the property that takes precedence over

	/// [MediaQueryData.platformBrightness] for descendant widgets.
	/// [MediaQueryData.platformBrightness] for descendant Material widgets.

	/// Retrieves the [Brightness] to use for descendant widgets, based
	/// Retrieves the [Brightness] to use for descendant Material widgets, based

	/// [MediaQuery] exists, instead of throwing.
	/// [MediaQuery] exists.

	/// * [brightnessOf], which throws if no valid [InheritedTheme] or
	/// * [brightnessOf], which returns a default value if no valid [InheritedTheme] or

-      await tester.pumpWidget(const SizedBox());
+      // Prevent the implicitly added View from adding a MediaQuery
+      await tester.pumpWidget(
+        RawView(view: FakeFlutterView(tester.view, viewId: 77), child: const SizedBox()),
+        wrapWithView: false,
+      );

Feat: Add brightnessOf method for theme #163733

Feat: Add brightnessOf method for theme #163733

Conversation

rkishan516 commented Feb 20, 2025

Pre-launch Checklist

rkishan516 commented Feb 20, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rkishan516 Feb 21, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

navaronbracke Feb 21, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

navaronbracke left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

navaronbracke left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

navaronbracke commented Feb 22, 2025

rkishan516 commented Feb 22, 2025

QuncCccccc left a comment

Choose a reason for hiding this comment

navaronbracke commented Mar 14, 2025

dkwingsmt commented Mar 14, 2025

navaronbracke commented Mar 14, 2025

dkwingsmt commented Mar 14, 2025 • edited Loading

navaronbracke commented Mar 14, 2025

rkishan516 commented Mar 14, 2025

rkishan516 commented Mar 26, 2025

dkwingsmt left a comment

Choose a reason for hiding this comment

MitchellGoodwin left a comment

Choose a reason for hiding this comment

rkishan516 Feb 21, 2025 •

edited

Loading

navaronbracke Feb 21, 2025 •

edited

Loading

dkwingsmt commented Mar 14, 2025 •

edited

Loading