-
Notifications
You must be signed in to change notification settings - Fork 28
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
CONV-1435: Gravity layout #498
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously reviewed in nearly this state and approved. It would be good to get someone else from UI Systems to weigh in, or even Kyle.
let targetY = self.contentOffset.y + heightDelta | ||
|
||
self.contentOffset.y = min(targetY, maxContentOffsetY) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add an assert
if .bottom
is provided, but the layout direction is .horizontal
? Just so folks know it's not supported.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- todo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice work – thank you for getting this across the line!
f11883f
to
94ac6f9
Compare
0643576
to
b4a0e45
Compare
- Adds a new Chat App demo and a new behavior called verticalLayoutGravity. When verticalLayoutGravity is set to bottom, scrolling works the way you would expect for a messaging app.
gravity-demo.mp4
Checklist
Please do the following before merging:
Main
section.