[ 
https://issues.apache.org/jira/browse/WEEX-653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16686308#comment-16686308
 ] 

ASF GitHub Bot commented on WEEX-653:
-------------------------------------

YorkShen commented on a change in pull request #1664: 
[WEEX-653][android][iOS][core] Total platform support rtl direction by CSS 
"direction:rtl"
URL: https://github.com/apache/incubator-weex/pull/1664#discussion_r233385041
 
 

 ##########
 File path: weex_core/Source/core/render/page/render_page.cpp
 ##########
 @@ -277,13 +278,16 @@ bool RenderPage::UpdateStyle(
                   flag = true;
               });
           break;
+          case kTypeInheritableLayout:
+              inheriableLayout = true;
+              break;
         default: break;
       }
     }
   }
 
   if (style != nullptr || margin != nullptr || padding != nullptr ||
-      border != nullptr)
+      border != nullptr || inheriableLayout)
 
 Review comment:
   Explain to me, thanks.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> feature for Rtl layout
> ----------------------
>
>                 Key: WEEX-653
>                 URL: https://issues.apache.org/jira/browse/WEEX-653
>             Project: Weex
>          Issue Type: New Feature
>          Components: Android, iOS, Web Renderer
>    Affects Versions: 0.18
>            Reporter: Kai Tian
>            Assignee: YorkShen
>            Priority: Major
>             Fix For: 0.18
>
>
> [android][ios][core] Whole 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



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to