diff --git a/docs/demos/views/list.vue b/docs/demos/views/list.vue
index 36f31ee20..3a5b50e96 100644
--- a/docs/demos/views/list.vue
+++ b/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;
diff --git a/packages/list/index.vue b/packages/list/index.vue
index 223ea2e46..c1f484ca1 100644
--- a/packages/list/index.vue
+++ b/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;
       }
 
diff --git a/packages/mixins/popup/index.js b/packages/mixins/popup/index.js
index 288a46ada..bca493b4f 100644
--- a/packages/mixins/popup/index.js
+++ b/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() {