-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P3Issues that are less important to the Flutter projectIssues that are less important to the Flutter projecta: typographyText rendering, possibly libtxtText rendering, possibly libtxtframeworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team
Description
See implementation of TextStyle.copyWith:
flutter/packages/flutter/lib/src/painting/text_style.dart
Lines 894 to 900 in a7aca4b
| String? newDebugLabel; | |
| assert(() { | |
| if (this.debugLabel != null) { | |
| newDebugLabel = debugLabel ?? '(${this.debugLabel}).copyWith'; | |
| } | |
| return true; | |
| }()); |
(where debugLabel is a copyWith param).
The debugLabel param is ignored unless the receiver (the TextStyle you're calling copyWith on) already has a debugLabel. This is problematic when you have a text style without a debug label and you want to make a copy that has a debug label.
gnprice and brunogc
Metadata
Metadata
Assignees
Labels
P3Issues that are less important to the Flutter projectIssues that are less important to the Flutter projecta: typographyText rendering, possibly libtxtText rendering, possibly libtxtframeworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-frameworkOwned by Framework teamOwned by Framework teamtriaged-frameworkTriaged by Framework teamTriaged by Framework team