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

[WEEX-653][android][iOS][core] Total platform support rtl direction by CSS "direction:rtl" #1664

Merged
merged 74 commits into from
Nov 15, 2018

Conversation

win80540
Copy link
Contributor

[android][ios][core] Total platform support rtl direction by CSS "direction:rtl"

[core] Update the new layout engine to support inheritable CSS "direction:rtl".
[ios][android] Update component layer, to support list, slide, scroller RTL direction. Update WXEnvironment, now we can get system layout direction by WXEnvironment.layoutDirection. Update WXDomModule, now we can get element layout direction by getLayoutDirection(ref,callback)
Few languages such as Arabic, Hebrew, or Persian are written from Right to Left, but weex not support RTL layouts. To handle them. Since this PR merged, we can use weex for RTL languages in both android and ios. And Scroller performance like native, layout and scroll direction will be reverse.

There are some test Demos:
Div+Scroller http://dotwe.org/vue/9c3ee9824dd49111a87a93ea6513039a
Horizontal Scroller http://dotwe.org/vue/0dd282e3183c2384ec38a465c915ce41
Scroller (ScrollToElement) http://dotwe.org/vue/a24001978199df6e981673e9460598c8
Slider http://dotwe.org/vue/7e45bcf71ad396032944bc51541d6350
Animation http://dotwe.org/vue/b578facdbb623f6922f0905ae09b7e49
envDirection + getLayoutDirection http://dotwe.org/vue/d6c5cf7a2a907c33cf4ba25fea6e6ef4

@weex-bot
Copy link

weex-bot commented Oct 24, 2018

Fails
🚫 Failed to run assembleDebug task for android.
Messages
📖 android build verification finished.

Generated by 🚫 dangerJS

android/sdk/build.gradle Outdated Show resolved Hide resolved
weex_core/Source/core/layout/layout.cpp Outdated Show resolved Hide resolved
weex_core/Source/core/layout/layout.cpp Outdated Show resolved Hide resolved
weex_core/Source/core/render/node/render_scroller.h Outdated Show resolved Hide resolved
@YorkShen YorkShen merged commit 3cb8e9e into apache:master Nov 15, 2018
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.

3 participants