Unverified Commit a1ea383f authored by Qun Cheng's avatar Qun Cheng Committed by GitHub

Label should always be aligned with text in filled input decoration (#115540)

Co-authored-by: 's avatarQun Cheng <quncheng@google.com>
parent 31f86316
...@@ -1555,14 +1555,14 @@ class _RenderDecoration extends RenderBox with SlottedContainerRenderObjectMixin ...@@ -1555,14 +1555,14 @@ class _RenderDecoration extends RenderBox with SlottedContainerRenderObjectMixin
case TextDirection.rtl: // origin is on the right case TextDirection.rtl: // origin is on the right
startX = labelOffset.dx + labelWidth * (1.0 - scale); startX = labelOffset.dx + labelWidth * (1.0 - scale);
floatStartX = startX; floatStartX = startX;
if (prefixIcon != null && !decoration.alignLabelWithHint) { if (prefixIcon != null && !decoration.alignLabelWithHint && isOutlineBorder) {
floatStartX += material3 ? _boxSize(prefixIcon).width - contentPadding.left : 0.0; floatStartX += material3 ? _boxSize(prefixIcon).width - contentPadding.left : 0.0;
} }
break; break;
case TextDirection.ltr: // origin on the left case TextDirection.ltr: // origin on the left
startX = labelOffset.dx; startX = labelOffset.dx;
floatStartX = startX; floatStartX = startX;
if (prefixIcon != null && !decoration.alignLabelWithHint) { if (prefixIcon != null && !decoration.alignLabelWithHint && isOutlineBorder) {
floatStartX += material3 ? -_boxSize(prefixIcon).width + contentPadding.left : 0.0; floatStartX += material3 ? -_boxSize(prefixIcon).width + contentPadding.left : 0.0;
} }
break; break;
......
...@@ -2842,6 +2842,25 @@ void main() { ...@@ -2842,6 +2842,25 @@ void main() {
expect(tester.getBottomLeft(find.text('text')).dx, 48.0); expect(tester.getBottomLeft(find.text('text')).dx, 48.0);
expect(getBorderWeight(tester), 2.0); expect(getBorderWeight(tester), 2.0);
}); });
testWidgets('Floating label for filled input decoration is aligned with text', (WidgetTester tester) async {
await tester.pumpWidget(
buildInputDecorator(
useMaterial3: useMaterial3,
decoration: const InputDecoration(
prefixIcon: Icon(Icons.ac_unit),
labelText: 'label',
filled: true,
),
isFocused: true,
),
);
expect(tester.getSize(find.byType(InputDecorator)), const Size(800.0, 56.0));
expect(tester.getTopLeft(find.text('label')).dx, 48.0);
expect(tester.getBottomLeft(find.text('text')).dx, 48.0);
expect(getBorderWeight(tester), 2.0);
});
}); });
group('3 point interpolation alignment', () { group('3 point interpolation alignment', () {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment