From 77892feeaa063e8f33c93001d930b33c3b5957b8 Mon Sep 17 00:00:00 2001
From: XPoet <i@xpoet.cn>
Date: Mon, 9 Nov 2020 10:02:41 +0800
Subject: [PATCH] perf: optimize 'side_tools' configuration item of the theme
 _config.yml

---
 layout/_partial/scripts.ejs      | 13 ++++++++-----
 layout/page.ejs                  |  2 +-
 scripts/helpers/export-config.js |  3 ++-
 source/css/layout/page.styl      |  2 +-
 source/js/dark-light-toggle.js   |  3 +--
 source/js/utils.js               |  5 +++--
 6 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/layout/_partial/scripts.ejs b/layout/_partial/scripts.ejs
index 0c85c9c..0012acf 100644
--- a/layout/_partial/scripts.ejs
+++ b/layout/_partial/scripts.ejs
@@ -1,16 +1,19 @@
 <%- js([
     'js/utils.js',
     'js/header-shrink.js',
-    'js/dark-light-toggle.js',
     'js/main.js'
 ]) %>
 
-<% if (theme.local_search.enable && config.search) { %>
-    <%- js('js/local-search.js') %>
+<% if (theme.side_tools.enable) { %>
+    <%- js('js/dark-light-toggle.js') %>
+
+    <% if (theme.back2top.enable) { %>
+        <%- js('js/back2top.js') %>
+    <% } %>
 <% } %>
 
-<% if (theme.back2top.enable) { %>
-    <%- js('js/back2top.js') %>
+<% if (theme.local_search.enable && config.search) { %>
+    <%- js('js/local-search.js') %>
 <% } %>
 
 <% if (is_post()) { %>
diff --git a/layout/page.ejs b/layout/page.ejs
index d8b4acc..38c2e88 100644
--- a/layout/page.ejs
+++ b/layout/page.ejs
@@ -56,7 +56,7 @@
     </div>
 
     <% if (theme.side_tools.enable) { %>
-        <div class="sidebar-tools-2">
+        <div class="bottom-sidebar-tools">
             <%- partial('_partial/tools2') %>
         </div>
     <% } %>
diff --git a/scripts/helpers/export-config.js b/scripts/helpers/export-config.js
index de86195..4a9d446 100644
--- a/scripts/helpers/export-config.js
+++ b/scripts/helpers/export-config.js
@@ -15,7 +15,8 @@ hexo.extend.helper.register('export_config', function () {
     localsearch: theme.local_search,
     codeblock: theme.codeblock,
     toc: theme.toc,
-    back2top: theme.back2top
+    back2top: theme.back2top,
+    side_tools: theme.side_tools
   };
   if (config.search) {
     exportConfig.path = config.search.path;
diff --git a/source/css/layout/page.styl b/source/css/layout/page.styl
index b34c0da..78ac963 100644
--- a/source/css/layout/page.styl
+++ b/source/css/layout/page.styl
@@ -166,7 +166,7 @@ $page-aside-width = 258px;
   }
 
 
-  .sidebar-tools-2 {
+  .bottom-sidebar-tools {
     position: fixed;
     bottom: $component-interspace;
     right: $component-interspace;
diff --git a/source/js/dark-light-toggle.js b/source/js/dark-light-toggle.js
index ee7c494..6ecd4dd 100644
--- a/source/js/dark-light-toggle.js
+++ b/source/js/dark-light-toggle.js
@@ -56,7 +56,7 @@ window.addEventListener('DOMContentLoaded', () => {
     },
 
     initModeToggleButton() {
-      this.modeToggleButton_dom && this.modeToggleButton_dom.addEventListener('click', () => {
+     this.modeToggleButton_dom.addEventListener('click', () => {
         if (window.matchMedia && window.matchMedia('(prefers-color-scheme: dark)').matches) {
           this.setItemUtil('light-mode', 'dark');
         } else {
@@ -64,7 +64,6 @@ window.addEventListener('DOMContentLoaded', () => {
         }
       });
     }
-
   }
 
   ILS.utils.modeToggle.initModeStatus();
diff --git a/source/js/utils.js b/source/js/utils.js
index 8ffa9cf..837185d 100644
--- a/source/js/utils.js
+++ b/source/js/utils.js
@@ -67,12 +67,13 @@ ILS.utils = {
 
   // tools
   registerToolsButtonClick() {
+    if (!CONFIG.side_tools.enable) return;
 
-    let isOpen = false
+    let isOpen = false;
     this.toolsMenuButton_dom = document.querySelector('.tools-button');
     this.toolsWrapperList_dom = document.querySelectorAll('.tools-wrapper li');
 
-    this.toolsMenuButton_dom && this.toolsMenuButton_dom.addEventListener('click', e => {
+    this.toolsMenuButton_dom.addEventListener('click', e => {
       isOpen = !isOpen;
 
       const toolsMenuButtonIcon = this.toolsMenuButton_dom.querySelector('i');