-
Notifications
You must be signed in to change notification settings - Fork 540
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
[Android] Fix height: stretch elements in columns #5085
Conversation
ContainerStyle parentContainerStyle = renderArgs.getContainerStyle(); | ||
ContainerStyle styleForThis = ContainerRenderer.GetLocalContainerStyle(columnSet, parentContainerStyle); | ||
|
||
for (int i = 0; i < columnVectorSize; i++) | ||
{ | ||
Column column = columnVector.get(i); | ||
|
||
if(columnSet.GetMinHeight() > column.GetMinHeight()) { |
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.
{ [](start = 65, length = 1)
[nit] for style consistency with the rest of this code let's put this brace on the next line
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.
Hi @golddove. This non-spec pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along. |
Hi @golddove; Thanks for taking action on your previously stale pull request. Resetting staleness. |
* ColumnRenderer back to LinearLayout * Reuse function * ColumnSet minHeight * Address review comment, refactor, remove extraneous empty view
Related Issue
Fixes #4989
Description
Elements that use
LinearLayout.LayoutParams
to set weight for vertical stretch fail when those elements are laid out in a FlexboxLayout. There is actually no reason for ColumnRenderer to use flexbox, it just needs the right flex properties set so the ColumnSetRenderer flexbox can correctly layout the child columns. So, reverting to use LinearLayout for ColumnRenderer.Sample Card
(most samples were modified for these screenshots by shortening the text)
v1.1\Tests\ColumnSet*
v1.1\Tests\Column.VerticalAlignment
v1.2\Elements\Column.MinHeight
v1.2\Elements\ColumnSet.MinHeight
How Verified
Screenshots above
Microsoft Reviewers: Open in CodeFlow