Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't update contentPaddingBottom if container is 0 height #76

Merged
merged 1 commit into from
May 12, 2020

Conversation

jscheel
Copy link
Contributor

@jscheel jscheel commented May 12, 2020

This addresses a bug where we calculate the padding bottom when the
container hasn't actually rendered properly yet. This is a hacky solution
that will need to be replaced once we fix the sizing a bit better later.

This addresses a bug where we calculate the padding bottom when the
container hasn't actually rendered properly yet. This is a hacky solution
that will need to be replaced once we fix the sizing a bit better later.
@jscheel jscheel added bug Something isn't working internal Changes only affect the internal API patch Increment the patch version when merged release Create a release when this pr is merged labels May 12, 2020
@jscheel jscheel merged commit da1d2b5 into master May 12, 2020
@jscheel jscheel deleted the fix-scroll-padding-fixer branch May 12, 2020 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working internal Changes only affect the internal API patch Increment the patch version when merged release Create a release when this pr is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant