material-components-android icon indicating copy to clipboard operation
material-components-android copied to clipboard

[ShapeableImageView] Fix multiple contentPadding issues

Open jpmcosta opened this issue 3 years ago • 1 comments

Solution

Closes #2063.

Should fix the following contentPadding issues:

  • contentPadding getters returning outdated values when app:contentPaddingStart or app:contentPaddingEnd are defined in xml;
  • contentPadding being applied twice, if setContentPadding or setContentPaddingRelative are called before the view is measured;
    • in this case, the shape mask is also affected to have padding even if only contentPadding is defined.

jpmcosta avatar Mar 21 '22 21:03 jpmcosta

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

google-cla[bot] avatar Mar 21 '22 21:03 google-cla[bot]