fix error

Author: WangRongsheng

_includes/head.html

@@ -20,7 +20,7 @@
     <link rel="shortcut icon" href="{{ site.icon | prepend: '/assets/img/' | relative_url}}"/>
     {% endif %}
     <script src="{{ '/assets/js/block_polyfill_io.js' | relative_url }}"></script>
-    <script src="{{ '/assets/js/wechat_popover.js' | relative_url }}" defer></script>
+    <script src="{{ '/assets/js/wechat_popover.js?v=2' | relative_url }}" defer></script>
     <link rel="stylesheet" href="{{ '/assets/css/main.css' | relative_url }}">
     <link rel="canonical" href="{{ page.url | replace:'index.html','' | absolute_url }}">
 

_includes/social.html

@@ -52,7 +52,7 @@
                      alt="Wechat" 
                      style="width:56px; height:56px; vertical-align:middle;">
               </button>
-              <span class="wechat-qr" role="dialog" aria-label="Wechat QR code">
+              <span class="wechat-qr" role="dialog" aria-label="Wechat QR code" hidden>
                 <img src="{{ site.wechat }}" alt="Wechat QR code">
               </span>
             </span>

_sass/_base.scss

@@ -236,46 +236,6 @@ blockquote {
       }
     }
 
-    .wechat-popover {
-      display: inline-flex;
-      position: relative;
-      align-items: center;
-      vertical-align: middle;
-    }
-
-    .wechat-trigger {
-      padding: 0;
-      border: 0;
-      background: transparent;
-      cursor: pointer;
-      line-height: 1;
-    }
-
-    .wechat-qr {
-      position: absolute;
-      left: 50%;
-      bottom: calc(100% + 12px);
-      z-index: 20;
-      display: none;
-      width: 176px;
-      padding: 10px;
-      transform: translateX(-50%);
-      border: 1px solid var(--global-divider-color);
-      border-radius: 8px;
-      background: var(--global-bg-color);
-      box-shadow: 0 10px 30px rgba(0, 0, 0, 0.18);
-    }
-
-    .wechat-qr img {
-      display: block;
-      width: 100%;
-      height: auto;
-      border-radius: 4px;
-    }
-
-    .wechat-popover.is-open .wechat-qr {
-      display: block;
-    }
   }
 }
 
@@ -332,6 +292,51 @@ blockquote {
 
 // Social (bottom)
 
+.wechat-popover {
+  display: inline-flex;
+  position: relative;
+  align-items: center;
+  vertical-align: middle;
+}
+
+.wechat-trigger {
+  padding: 0;
+  border: 0;
+  background: transparent;
+  cursor: pointer;
+  line-height: 1;
+}
+
+.wechat-qr {
+  position: absolute;
+  left: 50%;
+  bottom: calc(100% + 12px);
+  z-index: 20;
+  display: none;
+  width: 176px;
+  padding: 10px;
+  transform: translateX(-50%);
+  border: 1px solid var(--global-divider-color);
+  border-radius: 8px;
+  background: var(--global-bg-color);
+  box-shadow: 0 10px 30px rgba(0, 0, 0, 0.18);
+}
+
+.wechat-qr[hidden] {
+  display: none !important;
+}
+
+.wechat-qr img {
+  display: block;
+  width: 100%;
+  height: auto;
+  border-radius: 4px;
+}
+
+.wechat-popover.is-open .wechat-qr {
+  display: block;
+}
+
 .social {
   text-align: center;
 

assets/js/wechat_popover.js

@@ -1,10 +1,23 @@
 (function () {
   function closeAll(except) {
     document.querySelectorAll('.wechat-popover.is-open').forEach(function (popover) {
-      if (popover !== except) popover.classList.remove('is-open');
+      if (popover === except) return;
+      popover.classList.remove('is-open');
+      var qr = popover.querySelector('.wechat-qr');
+      if (qr) qr.hidden = true;
     });
   }
 
+  function setOpen(popover, isOpen) {
+    popover.classList.toggle('is-open', isOpen);
+    var qr = popover.querySelector('.wechat-qr');
+    if (qr) qr.hidden = !isOpen;
+  }
+
+  document.querySelectorAll('.wechat-popover').forEach(function (popover) {
+    setOpen(popover, false);
+  });
+
   document.addEventListener('click', function (event) {
     var trigger = event.target.closest('.wechat-trigger');
     var activePopover = event.target.closest('.wechat-popover');
@@ -13,7 +26,7 @@
       event.preventDefault();
       var willOpen = !activePopover.classList.contains('is-open');
       closeAll(activePopover);
-      activePopover.classList.toggle('is-open', willOpen);
+      setOpen(activePopover, willOpen);
       return;
     }