Skip to content

DefaultTextStyle.merge should be emphasized #122180

New issue
New issue
Closed
#122182
Closed
DefaultTextStyle.merge should be emphasized#122180
#122182
Assignees
bleroux
Labels
a: typographyText rendering, possibly libtxtd: api docsIssues with https://api.flutter.dev/f: material designflutter/packages/flutter/material repository.found in release: 3.7Found to occur in 3.7found in release: 3.9Found to occur in 3.9frameworkflutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer version
@bleroux

Description

@bleroux
bleroux
opened on Mar 8, 2023

DefaultTextStyle default constructor might be error-prone because it overrides existing DefaultTextStyle, especially the one set by MaterialApp.

DefaultTextStyle.merge is probably what most users want to use. Adding an example in DefaultTextStyle documentation would be one solution to put more emphasis on DefaultTextStyle.merge.

Metadata

Metadata

Assignees

Labels

a: typographyText rendering, possibly libtxtd: api docsIssues with https://api.flutter.dev/f: material designflutter/packages/flutter/material repository.found in release: 3.7Found to occur in 3.7found in release: 3.9Found to occur in 3.9frameworkflutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer version

Type

No type

Projects

Nevercode

Status

Done (PR merged)

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions