Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

[core] fix flex-end in row-reverse #2321

Merged
merged 1 commit into from
May 5, 2019
Merged

Conversation

win80540
Copy link
Contributor

We fount that, when flex-direction is row-reverse, justify-content:flex-end is not available, not matter justify-content is flex-start or flex-end, layout system always layout items as flex-start.

Demo:
https://jsplayground.taobao.org/raxplayground/a3ad1d37-a6b7-41bb-9b72-3c00ab4ee161

@win80540 win80540 requested a review from YorkShen as a code owner April 15, 2019 09:42
@weex-bot
Copy link

weex-bot commented Apr 15, 2019

Messages
📖 No android file has been changed.
📖 android build verification finished.

Generated by 🚫 dangerJS

@gubaojian
Copy link
Member

@YorkShen layout preview

Copy link
Contributor

@YorkShen YorkShen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there may be something wrong.

weex_core/Source/core/layout/layout.cpp Show resolved Hide resolved
@win80540 win80540 requested a review from wqyfavor as a code owner May 5, 2019 08:03
@win80540 win80540 force-pushed the hotfix/flex_end branch from 21a29be to b175fa9 Compare May 5, 2019 08:05
Copy link
Contributor

@YorkShen YorkShen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks for your contribution.

@YorkShen YorkShen merged commit 1ff3ac1 into apache:master May 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants