Merge pull request #1190 from pjkaufman/master

Remove JS Styling, Minify CSS, Speed Up Settings Page Loading, Remove Use of `innerHTML`, and Remove Default Hotkey

.eslintignore

@@ -4,3 +4,4 @@ main.js
 translation-helper.js
 .eslintrc.js
 babel.config.js
+postss.config.js

.github/workflows/release.yml

@@ -20,15 +20,16 @@ jobs:
         with:
           node-version: "18.x"
 
-      - name: Build plugin
+      - name: Build plugin and minfy css
         run: |
           npm ci
           npm run build
+          npm run minify-css
 
       - name: Create Manual Install Zip
         run: |
           mkdir ${{ env.PLUGIN_NAME }}
-          cp main.js manifest.json src/styles.css src/utils/default-misspellings.md ${{ env.PLUGIN_NAME }}
+          cp main.js manifest.json styles.css src/utils/default-misspellings.md ${{ env.PLUGIN_NAME }}
           zip -r ${{ env.PLUGIN_NAME }}.zip ${{ env.PLUGIN_NAME }}
 
       - name: Create release
@@ -40,4 +41,4 @@ jobs:
           gh release create "$tag" \
             --title="$tag" \
             --draft \
-            main.js manifest.json src/styles.css ${{ env.PLUGIN_NAME }}.zip
+            main.js manifest.json styles.css ${{ env.PLUGIN_NAME }}.zip

.gitignore

@@ -31,6 +31,10 @@ test-vault/.obsidian/plugins/table-editor-obsidian/*
 # occasional bug fix test
 main.zip
 
+# ignore generated files
+main.js
+styles.css
+
 yarn.lock
 
 .DS_Store

esbuild.config.mjs

@@ -27,11 +27,11 @@ const mockedPlugins = [replace({
     // remove the use of obsidian in the options to allow for docs.js to run
     'import {Setting} from \'obsidian\';': '',
     // remove the use of obsidian in settings helper to allow for docs.js to run
-    'import {App, Component, MarkdownRenderer} from \'obsidian\';': '',
+    'import {App, MarkdownRenderer} from \'obsidian\';': '',
     // remove the use of obsidian in the auto-correct files picker to allow for docs.js to run
-    'import {Setting, Component, App, TFile, normalizePath, ExtraButtonComponent} from \'obsidian\';': '',
+    'import {Setting, App, TFile, normalizePath, ExtraButtonComponent} from \'obsidian\';': '',
     // remove the use of obsidian in add custom row to allow for docs.js to run
-    'import {App, Component, Setting} from \'obsidian\';': '',
+    'import {App, Setting} from \'obsidian\';': '',
     // remove the use of obsidian in suggest to allow for docs.js to run
     'import {App, ISuggestOwner, Scope} from \'obsidian\';': '',
     // remove the use of obsidian in md file suggester to allow for docs.js to run