Skip to content

Commit 5b24378

Browse files
author
yinxu
committed
Merge branch 'master' into fix/fix_slider_text_null_error
* master: plugin_ffi template comment fix (flutter#148378) Roll Flutter Engine from 942d7c35de75 to 9e17588b330c (2 revisions) (flutter#148455) Reland fix TextField helper top padding on M3 (flutter#146754) Removing duplicate assert on `VisualDensity` constructor (flutter#148281) Roll Flutter Engine from 65ac4bf96ed7 to 942d7c35de75 (1 revision) (flutter#148450) Roll Flutter Engine from c11d64be5102 to 65ac4bf96ed7 (3 revisions) (flutter#148448) Roll Flutter Engine from f6195e9d4b4b to c11d64be5102 (1 revision) (flutter#148445)
2 parents 6c74917 + 8a4179e commit 5b24378

File tree

6 files changed

+9
-11
lines changed

6 files changed

+9
-11
lines changed

bin/internal/engine.version

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
f6195e9d4b4bed13e144c7b551bd11aaa38fc4fb
1+
9e17588b330ca997f9989cfb63137425e74609b8

packages/flutter/lib/src/material/input_decorator.dart

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -729,7 +729,9 @@ class _RenderDecoration extends RenderBox with SlottedContainerRenderObjectMixin
729729
_expands = expands,
730730
_material3 = material3;
731731

732-
static const double subtextGap = 8.0;
732+
// TODO(bleroux): consider defining this value as a Material token and making it
733+
// configurable by InputDecorationTheme.
734+
double get subtextGap => material3 ? 4.0 : 8.0;
733735

734736
RenderBox? get icon => childForSlot(_DecorationSlot.icon);
735737
RenderBox? get input => childForSlot(_DecorationSlot.input);

packages/flutter/lib/src/material/theme_data.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2328,7 +2328,6 @@ class VisualDensity with Diagnosticable {
23282328
this.horizontal = 0.0,
23292329
this.vertical = 0.0,
23302330
}) : assert(vertical <= maximumDensity),
2331-
assert(vertical <= maximumDensity),
23322331
assert(vertical >= minimumDensity),
23332332
assert(horizontal <= maximumDensity),
23342333
assert(horizontal >= minimumDensity);

packages/flutter/test/material/input_decorator_test.dart

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4490,23 +4490,20 @@ void main() {
44904490
});
44914491

44924492
group('Material3 - InputDecoration helper/counter/error', () {
4493-
// Overall height for InputDecorator (filled or outlined) is 80dp on mobile:
4493+
// Overall height for InputDecorator (filled or outlined) is 76dp on mobile:
44944494
// 8 - top padding
44954495
// 12 - floating label (font size = 16 * 0.75, line height is forced to 1.0)
44964496
// 4 - gap between label and input
44974497
// 24 - input text (font size = 16, line height = 1.5)
44984498
// 8 - bottom padding
4499-
// 8 - gap above supporting text
4499+
// 4 - gap above helper/error/counter
45004500
// 16 - helper/counter (font size = 12, line height is 1.5)
45014501
const double topPadding = 8.0;
45024502
const double floatingLabelHeight = 12.0;
45034503
const double labelInputGap = 4.0;
45044504
const double inputHeight = 24.0;
45054505
const double bottomPadding = 8.0;
4506-
// TODO(bleroux): make the InputDecorator implementation compliant with M3 spec by changing
4507-
// the helperGap to 4.0 instead of 8.0.
4508-
// See https://github.com/flutter/flutter/issues/144984.
4509-
const double helperGap = 8.0;
4506+
const double helperGap = 4.0;
45104507
const double helperHeight = 16.0;
45114508
const double containerHeight = topPadding + floatingLabelHeight + labelInputGap + inputHeight + bottomPadding; // 56.0
45124509
const double fullHeight = containerHeight + helperGap + helperHeight; // 80.0 (should be 76.0 based on M3 spec)
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
// Relative import to be able to reuse the C sources.
2-
// See the comment in ../{projectName}}.podspec for more information.
2+
// See the comment in ../{{projectName}}.podspec for more information.
33
#include "../../src/{{projectName}}.c"
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
// Relative import to be able to reuse the C sources.
2-
// See the comment in ../{projectName}}.podspec for more information.
2+
// See the comment in ../{{projectName}}.podspec for more information.
33
#include "../../src/{{projectName}}.c"

0 commit comments

Comments
 (0)