New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecate useMaterial3
parameter in ThemeData.copyWith()
#131455
Merged
QuncCccccc
merged 5 commits into
flutter:master
from
QuncCccccc:deprecate_use_material_3_in_copy_with
Jul 28, 2023
Merged
Deprecate useMaterial3
parameter in ThemeData.copyWith()
#131455
QuncCccccc
merged 5 commits into
flutter:master
from
QuncCccccc:deprecate_use_material_3_in_copy_with
Jul 28, 2023
Conversation
This file contains 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
github-actions
bot
added
framework
flutter/packages/flutter repository. See also f: labels.
f: material design
flutter/packages/flutter/material repository.
c: tech-debt
Technical debt, code quality, testing, etc.
labels
Jul 28, 2023
guidezpl
approved these changes
Jul 28, 2023
Comment on lines
1640
to
1641
'Use ThemeData.useMaterial3 instead. ' | ||
'Setting this value cannot successfully change the default theme. ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested change
'Use ThemeData.useMaterial3 instead. ' | |
'Setting this value cannot successfully change the default theme. ' | |
'Use a ThemeData constructor (.from, .light, or .dark) instead. ' | |
'These constructors all have a useMaterial3 argument, and they set appropriate default values based on its value. See the useMaterial3 API documentation for full details.' |
@@ -25,6 +25,17 @@ | |||
# * TextTheme: fix_text_theme.yaml | |||
version: 1 | |||
transforms: | |||
# Changes made in https://github.com/flutter/flutter/pull/131455 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be moved to the bottom
QuncCccccc
force-pushed
the
deprecate_use_material_3_in_copy_with
branch
from
July 28, 2023 06:25
016f737
to
939e3db
Compare
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Jul 29, 2023
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/packages
that referenced
this pull request
Jul 29, 2023
auto-submit bot
pushed a commit
to flutter/packages
that referenced
this pull request
Jul 29, 2023
flutter/flutter@e81907e...c2b40ca 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from 24ccd8fffbcb to 5184062f7543 (2 revisions) (flutter/flutter#131554) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from 90bc483524fe to 24ccd8fffbcb (1 revision) (flutter/flutter#131540) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from 27128f29a066 to 90bc483524fe (2 revisions) (flutter/flutter#131535) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from 69b5b77edaf7 to 27128f29a066 (1 revision) (flutter/flutter#131533) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from b854fdea2715 to 69b5b77edaf7 (1 revision) (flutter/flutter#131532) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0a5b2f521539 to b854fdea2715 (1 revision) (flutter/flutter#131530) 2023-07-29 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 2.21.0 to 2.21.2 (flutter/flutter#131512) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from f5c17f0e2406 to 0a5b2f521539 (2 revisions) (flutter/flutter#131529) 2023-07-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from b4bf592279d6 to f5c17f0e2406 (1 revision) (flutter/flutter#131525) 2023-07-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from da3721adba65 to b4bf592279d6 (2 revisions) (flutter/flutter#131519) 2023-07-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from aa1278eb7b84 to da3721adba65 (5 revisions) (flutter/flutter#131514) 2023-07-28 leroux_bruno@yahoo.fr Reland - Fix floating SnackBar throws when FAB is on the top (flutter/flutter#131475) 2023-07-28 ian@hixie.ch Device discovery output cleanup (flutter/flutter#131223) 2023-07-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 182e1189873b to aa1278eb7b84 (2 revisions) (flutter/flutter#131500) 2023-07-28 tessertaha@gmail.com Update `BottomSheet.enableDrag` & `BottomSheet.showDragHandle` docs for animation controller (flutter/flutter#131484) 2023-07-28 36861262+QuncCccccc@users.noreply.github.com Deprecate `useMaterial3` parameter in `ThemeData.copyWith()` (flutter/flutter#131455) 2023-07-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 73615d66b2fe to 182e1189873b (2 revisions) (flutter/flutter#131492) 2023-07-28 tessertaha@gmail.com Add `CheckedPopupMenuItem�.labelTextStyle` and update default text style for Material 3 (flutter/flutter#131060) 2023-07-28 ychris@google.com Remove obsolete `complex_layout_ios__compile.dart` test (flutter/flutter#131487) 2023-07-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from e97014c71014 to 73615d66b2fe (3 revisions) (flutter/flutter#131485) 2023-07-28 chevalier.dan@gmail.com Shared state to support multi screen inspection (flutter/flutter#129452) 2023-07-28 engine-flutter-autoroll@skia.org Roll Packages from f4ae933 to 10aab44 (4 revisions) (flutter/flutter#131483) 2023-07-28 ychris@google.com Move ios_app_with_extensions_test to host only (flutter/flutter#131441) 2023-07-28 tessertaha@gmail.com Update `Card.color` documentation for Material 3 (flutter/flutter#131468) 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 rmistry@google.com,stuartmorgan@google.com,tarrinneal@google.com 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 To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
LouiseHsu
pushed a commit
to LouiseHsu/flutter
that referenced
this pull request
Jul 31, 2023
vashworth
pushed a commit
to vashworth/flutter
that referenced
this pull request
Aug 2, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
c: tech-debt
Technical debt, code quality, testing, etc.
f: material design
flutter/packages/flutter/material repository.
framework
flutter/packages/flutter repository. See also f: labels.
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.
This PR is to deprecate
useMaterial3
parameter inThemeData.copyWith()
.Setting
useMaterial3
to false inThemeData.copyWith()
doesn't force Flutter to use Material 2, instead, we should set it inThemeData()
directly. The documentation ofuseMaterial3
has covered this limitation: https://api.flutter.dev/flutter/material/ThemeData/useMaterial3.html, but it still likely to be misused in the.copyWith()
method.A related issue happens in b/292560771
Related to #131041
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.