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

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

Github user gurisxie commented on the issue:

    https://github.com/apache/incubator-weex/pull/844
  
    HI, @myeveryheart there is something wrong with your pr.
    
    It will not fill full when setViewport = 640.
    
    please have a look.
    
    the tc link:[viewport 
tc](https://github.com/apache/incubator-weex/blob/master/test/pages/modules/viewport-basic.vue)
    
    the result pic:
    
![image](https://user-images.githubusercontent.com/7436932/33058996-4dfe0eca-cecd-11e7-97fd-a9e71fbb7efc.png)


> The setViewport method in WXMetaModule is ASYN right now which makes 
> weexInstance.viewportWidth setted after view created.
> --------------------------------------------------------------------------------------------------------------------------
>
>                 Key: WEEX-99
>                 URL: https://issues.apache.org/jira/browse/WEEX-99
>             Project: Weex
>          Issue Type: Bug
>          Components: iOS
>            Reporter: myeveryheart
>            Assignee: Adam Feng
>         Attachments: 粘贴图片.png, 粘贴图片1.png, 粘贴图片2.png, 粘贴图片3.png
>
>
> The meta module can be used to declare meta information for a single page, 
> such as the viewport of the container.
> It provides a method 'setViewport' to change the viewport of the page.
> But the method is ASYN right now which makes weexInstance.viewportWidth 
> setted after view created.
> The bug is in iOS and Android.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to