Skip to content

Conversation

@bleroux
Copy link
Contributor

@bleroux bleroux commented Feb 27, 2023

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 f: material design flutter/packages/flutter/material repository. 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
Copy link
Contributor

@HansMuller HansMuller left a comment

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

f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

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