[bugfix] List: should not trigger load event when hidden (#751)

docs/demos/views/list.vue

@@ -3,7 +3,7 @@
     <demo-block :title="$t('basicUsage')">
       <p class="page-desc">{{ $t('text') }}</p>
       <van-pull-refresh v-model="refreshing" @refresh="onRefresh">
-        <van-list 
+        <van-list
           v-model="loading"
           :finished="finished"
           @load="onLoad"
@@ -39,7 +39,7 @@ export default {
     onLoad() {
       setTimeout(() => {
         for (let i = 0; i < 10; i++) {
-          let text = this.list.length + 1;
+          const text = this.list.length + 1;
           this.list.push(text < 10 ? '0' + text : text);
         }
         this.loading = false;

packages/list/index.vue

@@ -79,7 +79,7 @@ export default create({
       const scrollerHeight = utils.getVisibleHeight(scroller);
 
       /* istanbul ignore next */
-      if (!scrollerHeight) {
+      if (!scrollerHeight || utils.getComputedStyle(el).display === 'none') {
         return;
       }
 

packages/mixins/popup/index.js

@@ -46,15 +46,15 @@ export default {
       x: 0,
       y: 0
     };
-    if (this.value) {
-      this.open();
-    }
   },
 
   mounted() {
     if (this.getContainer) {
       this.move();
     }
+    if (this.value) {
+      this.open();
+    }
   },
 
   beforeDestroy() {