Skip to content

Fix Ink decoration image does not render - #121521

Merged
HansMuller merged 1 commit into
flutter:masterfrom
NevercodeHQ:fix_Ink_decoration_image_does_not_render
Feb 27, 2023
Merged

HansMuller merged 1 commit into
flutter:masterfrom
NevercodeHQ:fix_Ink_decoration_image_does_not_render

Conversation

@bleroux

@bleroux bleroux commented Feb 27, 2023 •

Copy link
Copy Markdown
Contributor

Description

This PR fixes an issue related to how the Ink widget is sized when there is only one dimension specified.
Ink used to rely on empty Container() (to expand on the unspecified directions), after #108672, Container() was replaced with SizedBox().

This PR uses a ConstrainedBox, which is more verbose but more accurate.

Related Issue

Fixes #119965

Tests

Adds 1 tests.
Splits and updates 1 existing test in two tests.

@flutter-dashboard flutter-dashboard Bot added p: material_ui material_ui package in flutter/packages framework flutter/packages/flutter repository. See also f: labels. labels Feb 27, 2023
@bleroux
bleroux requested a review from HansMuller February 27, 2023 13:51

@HansMuller HansMuller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix! LGTM

@HansMuller
HansMuller merged commit a297cb3 into flutter:master Feb 27, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 27, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 27, 2023
@bleroux
bleroux deleted the fix_Ink_decoration_image_does_not_render branch February 27, 2023 17:36
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 27, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 27, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 27, 2023
XilaiZhang added a commit that referenced this pull request Feb 28, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flutter 3.7: Column -> Material -> Ink -> Decoration -> AssetImage : Doesn't render image

2 participants