[GitHub] incubator-weex issue #830: [jsfm] Support receive tasks in weex-js-runtime

2017-10-31 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/830
  




  

  
  Fails

  
  
  :no_entry_sign:
  Code file html5/runtime/bridge/receiver.js does not have the 
copyright header.

  




  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  

  :warning:
  These new JS files do not have Flow enabled: 
html5/runtime/bridge/receiver.js

  

  :warning:
  This PR should update related documents as well. 

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #830: [jsfm] Support receive tasks in weex-js-ru...

2017-10-31 Thread Hanks10100
GitHub user Hanks10100 opened a pull request:

https://github.com/apache/incubator-weex/pull/830

[jsfm] Support receive tasks in weex-js-runtime

Move the `callJS` and `receiveTasks` into a separate file. This feature has 
no breaking change.

By support the `receiveTasks` in `weex-js-runtime`, this method can be 
removed from `weex-vue-framework` and `weex-rax-framework`. It can keep the 
js-native communication have the same behavior.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/Hanks10100/incubator-weex jsfm-feature-calljs

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/830.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #830


commit b9c49fd6e52282d1937814506e43b33189189f56
Author: Hanks 
Date:   2017-10-31T07:04:41Z

+ [jsfm] support receive tasks in js runtime




---


[1/2] incubator-weex git commit: onVisibilityChanged most sense right. but when native component custom set parent visibility, the method not paired on some platform, e.g when parent become invisible,

2017-10-31 Thread gurisxie
Repository: incubator-weex
Updated Branches:
  refs/heads/release-0.16 d50ca8300 -> 355800d99


onVisibilityChanged most sense right. but when native component custom set 
parent visibility, the method not paired on some platform, e.g when parent 
become invisible, changedView is this, visibility is invisible.


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/d9a75a76
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/d9a75a76
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/d9a75a76

Branch: refs/heads/release-0.16
Commit: d9a75a7660abf0ba41eeac13ddbcc168f55b1310
Parents: 094a9ea
Author: jianbai.gbj 
Authored: Wed Nov 1 11:50:20 2017 +0800
Committer: jianbai.gbj 
Committed: Wed Nov 1 11:50:20 2017 +0800

--
 .../main/java/com/taobao/weex/ui/view/WXImageView.java | 13 +
 1 file changed, 1 insertion(+), 12 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/d9a75a76/android/sdk/src/main/java/com/taobao/weex/ui/view/WXImageView.java
--
diff --git a/android/sdk/src/main/java/com/taobao/weex/ui/view/WXImageView.java 
b/android/sdk/src/main/java/com/taobao/weex/ui/view/WXImageView.java
index 00cb74a..c3daf53 100644
--- a/android/sdk/src/main/java/com/taobao/weex/ui/view/WXImageView.java
+++ b/android/sdk/src/main/java/com/taobao/weex/ui/view/WXImageView.java
@@ -192,18 +192,7 @@ public class WXImageView extends ImageView implements 
WXGestureObservable,
   }
 }
   }
-
-  @Override
-  protected void onVisibilityChanged(@NonNull View changedView, int 
visibility) {
-super.onVisibilityChanged(changedView, visibility);
-if(changedView == this){
-if(visibility == View.VISIBLE){
-  autoRecoverImage();
-}else{
-  autoReleaseImage();
-}
-}
-  }
+  
 
   @Override
   protected void onAttachedToWindow() {



[GitHub] incubator-weex pull request #829: custom bugfix when native custom component...

2017-10-31 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/829


---


[2/2] incubator-weex git commit: * [android] custom bugfix when native custom component set visibility

2017-10-31 Thread gurisxie
* [android] custom bugfix when native custom component set visibility


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/355800d9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/355800d9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/355800d9

Branch: refs/heads/release-0.16
Commit: 355800d9910fb568ad0108d7f2dfeb24c2433f64
Parents: d50ca83 d9a75a7
Author: gurisxie <279483...@qq.com>
Authored: Wed Nov 1 13:35:47 2017 +0800
Committer: gurisxie <279483...@qq.com>
Committed: Wed Nov 1 13:36:21 2017 +0800

--
 .../main/java/com/taobao/weex/ui/view/WXImageView.java | 13 +
 1 file changed, 1 insertion(+), 12 deletions(-)
--




[GitHub] incubator-weex issue #829: custom bugfix when native custom component set vi...

2017-10-31 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/829
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  

  :warning:
  This PR should update related documents as well. 

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #829: custom bugfix when native custom component...

2017-10-31 Thread gubaojian
GitHub user gubaojian opened a pull request:

https://github.com/apache/incubator-weex/pull/829

custom bugfix when native custom component  set visibility

onVisibilityChanged most sense right. but when native component custom set 
parent visibility, the method not paired on some platform, e.g when parent 
become invisible, changedView is this, visibility is invisible.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gubaojian/incubator-weex 
release-0.16-image-auto-recycle

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/829.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #829


commit d9a75a7660abf0ba41eeac13ddbcc168f55b1310
Author: jianbai.gbj 
Date:   2017-11-01T03:50:20Z

onVisibilityChanged most sense right. but when native component custom set 
parent visibility, the method not paired on some platform, e.g when parent 
become invisible, changedView is this, visibility is invisible.




---


[2/4] incubator-weex git commit: Merge branch 'release-0.16' of https://github.com/apache/incubator-weex into release-0.16

2017-10-31 Thread gurisxie
Merge branch 'release-0.16' of https://github.com/apache/incubator-weex into 
release-0.16


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/1538690b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/1538690b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/1538690b

Branch: refs/heads/release-0.16
Commit: 1538690b1f0a4b5848f7de6384f0ee54bd42b26c
Parents: be723e2 ee83fd3
Author: jianbai.gbj 
Authored: Wed Oct 25 22:37:12 2017 +0800
Committer: jianbai.gbj 
Committed: Wed Oct 25 22:37:12 2017 +0800

--
 .../com/taobao/weex/ui/component/WXComponent.java| 10 ++
 .../java/com/taobao/weex/utils/BoxShadowUtil.java| 15 +++
 2 files changed, 25 insertions(+)
--




[3/4] incubator-weex git commit: Merge branch 'release-0.16' of https://github.com/apache/incubator-weex into release-0.16

2017-10-31 Thread gurisxie
Merge branch 'release-0.16' of https://github.com/apache/incubator-weex into 
release-0.16


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/094a9ea0
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/094a9ea0
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/094a9ea0

Branch: refs/heads/release-0.16
Commit: 094a9ea04bc1a6f4fe8e4893751aca763121b42c
Parents: 1538690 5c3fffc
Author: jianbai.gbj 
Authored: Mon Oct 30 13:57:49 2017 +0800
Committer: jianbai.gbj 
Committed: Mon Oct 30 13:57:49 2017 +0800

--
 .../com/taobao/weex/bridge/WXBridgeManager.java | 17 +++-
 .../java/com/taobao/weex/common/Constants.java  |  1 +
 .../taobao/weex/ui/component/WXComponent.java   |  8 ++
 .../com/taobao/weex/ui/component/WXImage.java   | 30 ++
 .../com/taobao/weex/ui/view/WXImageView.java| 98 
 5 files changed, 149 insertions(+), 5 deletions(-)
--




[1/4] incubator-weex git commit: when cell amount is big, watch appear will none changed will result too many logs when scroll, move log to appear changed

2017-10-31 Thread gurisxie
Repository: incubator-weex
Updated Branches:
  refs/heads/release-0.16 672800f51 -> d50ca8300


when cell amount is big, watch appear will none changed will result too many 
logs when scroll, move log to appear changed


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/be723e2e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/be723e2e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/be723e2e

Branch: refs/heads/release-0.16
Commit: be723e2ede0732341bf3386829e19b6cbaa8fdbd
Parents: ae011d3
Author: jianbai.gbj 
Authored: Wed Oct 25 13:01:26 2017 +0800
Committer: jianbai.gbj 
Committed: Wed Oct 25 13:01:26 2017 +0800

--
 .../com/taobao/weex/ui/component/list/BasicListComponent.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/be723e2e/android/sdk/src/main/java/com/taobao/weex/ui/component/list/BasicListComponent.java
--
diff --git 
a/android/sdk/src/main/java/com/taobao/weex/ui/component/list/BasicListComponent.java
 
b/android/sdk/src/main/java/com/taobao/weex/ui/component/list/BasicListComponent.java
index c04ed0a..bf51870 100644
--- 
a/android/sdk/src/main/java/com/taobao/weex/ui/component/list/BasicListComponent.java
+++ 
b/android/sdk/src/main/java/com/taobao/weex/ui/component/list/BasicListComponent.java
@@ -1170,12 +1170,12 @@ public abstract class BasicListComponent

[GitHub] incubator-weex pull request #820: when cell amount is big, watch appear will...

2017-10-31 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/820


---


[4/4] incubator-weex git commit: * [android] when cell amount is big, watch appear will none changed will result too many logs

2017-10-31 Thread gurisxie
* [android] when cell amount is big, watch appear will none changed will result 
too many logs


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/d50ca830
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/d50ca830
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/d50ca830

Branch: refs/heads/release-0.16
Commit: d50ca83007003de3fa54627c86cde8eb854dbed7
Parents: 672800f 094a9ea
Author: gurisxie <279483...@qq.com>
Authored: Tue Oct 31 22:56:31 2017 +0800
Committer: gurisxie <279483...@qq.com>
Committed: Tue Oct 31 22:57:21 2017 +0800

--
 .../com/taobao/weex/ui/component/list/BasicListComponent.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
--




[2/2] incubator-weex git commit: * [html5] bugfix

2017-10-31 Thread gurisxie
* [html5] bugfix


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/672800f5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/672800f5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/672800f5

Branch: refs/heads/release-0.16
Commit: 672800f51d3eec6650fe575c66cbd74d06241f37
Parents: 5c3fffc cd84c57
Author: gurisxie <279483...@qq.com>
Authored: Tue Oct 31 22:54:23 2017 +0800
Committer: gurisxie <279483...@qq.com>
Committed: Tue Oct 31 22:54:56 2017 +0800

--
 html5/render/browser/extend/api/storage.js  | 123 ---
 html5/render/browser/extend/api/stream.js   |   9 +-
 html5/render/vue/README.md  |  20 ++-
 html5/render/vue/components/a.js|   3 -
 html5/render/vue/components/image.js|  42 +--
 html5/render/vue/components/slider/indicator.js |   4 +-
 html5/render/vue/env/weex.js|  34 -
 html5/render/vue/index.js   |   2 +-
 html5/render/vue/mixins/base.js |  81 +++-
 html5/render/vue/mixins/style.js|  22 +---
 html5/render/vue/modules/dom.js |   6 +-
 html5/render/vue/utils/component.js |  11 +-
 html5/render/vue/utils/lazyload.js  |  24 +++-
 html5/test/render/vue/components/image.js   |   1 -
 html5/test/render/vue/components/switch.js  |   9 +-
 html5/test/render/vue/components/text.js|   2 +-
 html5/test/render/vue/modules/dom.js|   1 +
 package.json|   2 +-
 packages/weex-vue-render/README.md  |  20 ++-
 packages/weex-vue-render/package.json   |   2 +-
 20 files changed, 313 insertions(+), 105 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/672800f5/html5/render/browser/extend/api/stream.js
--



[GitHub] incubator-weex pull request #827: * [html5] bugfix

2017-10-31 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-weex/pull/827


---


[1/2] incubator-weex git commit: * [html5] bugfix: - scroll event listenning on window. - img lazyloading. - scrollToElement on latest chrome. - rm id of a comp. - fix stream.fetch. - add statistics f

2017-10-31 Thread gurisxie
Repository: incubator-weex
Updated Branches:
  refs/heads/release-0.16 5c3fffcf6 -> 672800f51


* [html5] bugfix:
  - scroll event listenning on window.
  - img lazyloading.
  - scrollToElement on latest chrome.
  - rm id of a comp.
  - fix stream.fetch.
  - add statistics for components and modules.
  - support lazyload and appear watcher when body height set to 100%
  - add try catch for accessing localStorage may throw a error.
  - image support sprite (sprite-src, sprite-position, sprite-width).
  - fix indicator when there's only one slide in slider.


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/cd84c579
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/cd84c579
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/cd84c579

Branch: refs/heads/release-0.16
Commit: cd84c579bcc4e7c1a80a5b7d2c6c46a051e19649
Parents: aab1d74
Author: MrRaindrop 
Authored: Tue Sep 26 15:56:51 2017 +0800
Committer: MrRaindrop 
Committed: Tue Oct 31 21:26:31 2017 +0800

--
 html5/render/browser/extend/api/storage.js  | 123 ---
 html5/render/browser/extend/api/stream.js   |   9 +-
 html5/render/vue/README.md  |  20 ++-
 html5/render/vue/components/a.js|   3 -
 html5/render/vue/components/image.js|  42 +--
 html5/render/vue/components/slider/indicator.js |   4 +-
 html5/render/vue/env/weex.js|  34 -
 html5/render/vue/index.js   |   2 +-
 html5/render/vue/mixins/base.js |  81 +++-
 html5/render/vue/mixins/style.js|  22 +---
 html5/render/vue/modules/dom.js |   6 +-
 html5/render/vue/utils/component.js |  11 +-
 html5/render/vue/utils/lazyload.js  |  24 +++-
 html5/test/render/vue/components/image.js   |   1 -
 html5/test/render/vue/components/switch.js  |   9 +-
 html5/test/render/vue/components/text.js|   2 +-
 html5/test/render/vue/modules/dom.js|   1 +
 package.json|   2 +-
 packages/weex-vue-render/README.md  |  20 ++-
 packages/weex-vue-render/package.json   |   2 +-
 20 files changed, 313 insertions(+), 105 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/cd84c579/html5/render/browser/extend/api/storage.js
--
diff --git a/html5/render/browser/extend/api/storage.js 
b/html5/render/browser/extend/api/storage.js
index bdd4c22..cb4de3b 100644
--- a/html5/render/browser/extend/api/storage.js
+++ b/html5/render/browser/extend/api/storage.js
@@ -20,12 +20,33 @@
 
 'use strict'
 
-const supportLocalStorage = typeof localStorage !== 'undefined'
+let supportLocalStorage = false
+try {
+  supportLocalStorage = typeof localStorage !== 'undefined'
+}
+catch (err) {
+  // not support.
+}
+
 const SUCCESS = 'success'
 const FAILED = 'failed'
 const INVALID_PARAM = 'invalid_param'
 const UNDEFINED = 'undefined'
 
+function callFail (sender, callbackId, errorMsg) {
+  sender.performCallback(callbackId, {
+result: FAILED,
+data: errorMsg || UNDEFINED
+  })
+}
+
+function callNotSupportFail (sender, callbackId) {
+  sender.performCallback(callbackId, {
+result: FAILED,
+data: 'localStorage is disabled or not supported.'
+  })
+}
+
 const storage = {
 
   /**
@@ -36,11 +57,10 @@ const storage = {
* @param {function} callbackId
*/
   setItem: function (key, value, callbackId) {
+const sender = this.sender
 if (!supportLocalStorage) {
-  console.error('your browser is not support localStorage yet.')
-  return
+  return callNotSupportFail(sender, callbackId)
 }
-const sender = this.sender
 if (!key || (!value && value !== 0)) {
   sender.performCallback(callbackId, {
 result: 'failed',
@@ -57,10 +77,7 @@ const storage = {
 }
 catch (e) {
   // accept any exception thrown during a storage attempt as a quota error
-  sender.performCallback(callbackId, {
-result: FAILED,
-data: UNDEFINED
-  })
+  callFail(sender, callbackId)
 }
   },
 
@@ -70,11 +87,10 @@ const storage = {
* @param {function} callbackId
*/
   getItem: function (key, callbackId) {
+const sender = this.sender
 if (!supportLocalStorage) {
-  console.error('your browser is not support localStorage yet.')
-  return
+  return callNotSupportFail(sender, callbackId)
 }
-const sender = this.sender
 if (!key) {
   sender.performCallback(callbackId, {
 result: FAILED,
@@ -82,11 +98,17 @@ const storage = {
   })
   return
 }
-const val = 

[GitHub] incubator-weex issue #828: * [devtool] fix devtool network mode

2017-10-31 Thread weex-bot
Github user weex-bot commented on the issue:

https://github.com/apache/incubator-weex/pull/828
  





  

  
  Warnings

  
  
  :warning:
  No Changelog changes!

  

  :warning:
  This PR should update related documents as well. 

  




  

  
  Messages

  
  
  :book:
  danger test finished.

  




  Generated by :no_entry_sign: http://github.com/danger/danger-js/;>dangerJS




---


[GitHub] incubator-weex pull request #828: * [devtool] fix devtool network mode

2017-10-31 Thread atomtong
GitHub user atomtong opened a pull request:

https://github.com/apache/incubator-weex/pull/828

* [devtool] fix devtool network mode

* [devtool] fix devtool network mode

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/atomtong/incubator-weex 
release-0.16-devtool-bugfixed

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-weex/pull/828.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #828


commit 41221fc2c2e114d12c5256660d02deda1f842b05
Author: atomtong 
Date:   2017-10-31T09:27:08Z

* [devtool] fix devtool network mode




---