[Chips] In MDCChipField, account for horizontal padding when calculat…

…ing available width for text field (#9841) This change addresses an issue introduced in #9837, where the calculation of available width for the text field did not take into account the chip field's horizontal insets.
material-components · Mar 5, 2020 · 2ceeb0a · 2ceeb0a
1 parent 2ca4e3e
commit 2ceeb0a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/components/Chips/src/MDCChipField.m b/components/Chips/src/MDCChipField.m
@@ -734,7 +734,7 @@ - (CGFloat)availableWidthForTextInput {
   }
 
   CGRect lastChipFrame = [chipFrames.lastObject CGRectValue];
-  return boundsWidth - CGRectGetMaxX(lastChipFrame);
+  return boundsWidth - CGRectGetMaxX(lastChipFrame) - self.contentEdgeInsets.right;
 }
 
 // The width of the text input + the clear button.

diff --git a/...ns_64/MDCChipFieldSnapshotTests/testTextFieldIsOnLineBelowWideChips_11_2@2x.png b/...ns_64/MDCChipFieldSnapshotTests/testTextFieldIsOnLineBelowWideChips_11_2@2x.png
diff --git a/...MDCChipFieldSnapshotTests/testTextFieldIsOnTheSameLineAsNarrowChips_11_2@2x.png b/...MDCChipFieldSnapshotTests/testTextFieldIsOnTheSameLineAsNarrowChips_11_2@2x.png
diff --git a/...pshotTests/testTextFieldWithZeroMinimumWidthIsOnSameLineAsWideChips_11_2@2x.png b/...pshotTests/testTextFieldWithZeroMinimumWidthIsOnSameLineAsWideChips_11_2@2x.png