Skip to content

Conversation

@emakar
Copy link

@emakar emakar commented Feb 23, 2021

I needed BottomSheetBehavior to work with top margin being set to its view.
Currently this is buggy and gives various card jumps.
I also need this behavior with fitToContents = true

@google-cla google-cla bot added the cla: yes label Feb 23, 2021
@emakar emakar force-pushed the bottom-sheet-top-margin branch 2 times, most recently from a5d8307 to d2c9480 Compare February 23, 2021 14:41
so that it contributes to expanded offset
@emakar emakar force-pushed the bottom-sheet-top-margin branch from d2c9480 to 5c7819b Compare February 23, 2021 15:23
@emakar emakar changed the title [BottomSheetBehavior] add minFitToContentsOffset [BottomSheetBehavior] specially treat child's top margin Feb 23, 2021
@emakar
Copy link
Author

emakar commented Feb 23, 2021

I've done more testing and left only special top margin treatment

This is ready for review

@Lingviston
Copy link

Lingviston commented May 6, 2023

Any update on this one? It looks like it might also fix the issue with that standard marginTopSystemWindowInsets=true attribute. See #3380

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants