From d5b0abb2d9644c4b9326d4d95bdce11b1cca83be Mon Sep 17 00:00:00 2001 From: MickaelK Date: Mon, 26 Feb 2024 00:23:39 +1100 Subject: [PATCH] fix (public): linter in frontend rewrite --- public/assets/boot/ctrl_boot_frontoffice.js | 4 +- public/assets/components/breadcrumb.js | 31 +- .../components/decorator_shell_filemanager.js | 4 +- public/assets/components/dropdown.js | 2 +- public/assets/components/loader.js | 2 +- public/assets/components/menubar.js | 3 +- public/assets/helpers/loader.js | 2 +- public/assets/lib/animate.js | 3 +- public/assets/lib/assert.js | 6 +- public/assets/lib/chromecast.js | 18 +- public/assets/lib/polyfill.js | 3 - .../ctrl_backend_component_storage.js | 2 +- .../pages/adminpage/ctrl_backend_state.js | 2 +- .../assets/pages/adminpage/ctrl_settings.js | 2 +- public/assets/pages/connectpage/ctrl_form.js | 4 +- public/assets/pages/ctrl_error.js | 1 - public/assets/pages/ctrl_filespage.js | 4 +- public/assets/pages/ctrl_viewerpage.js | 5 +- .../assets/pages/filespage/ctrl_filesystem.js | 19 +- .../pages/filespage/ctrl_filesystem_state.js | 2 +- public/assets/pages/filespage/ctrl_submenu.js | 10 +- public/assets/pages/filespage/thing.js | 12 +- .../assets/pages/viewerpage/application_3d.js | 10 +- .../pages/viewerpage/application_audio.js | 33 +- .../pages/viewerpage/application_ebook.js | 16 +- .../pages/viewerpage/application_editor.js | 20 +- .../viewerpage/application_editor/clike.js | 4 +- .../viewerpage/application_editor/clojure.js | 4 +- .../viewerpage/application_editor/cmake.js | 4 +- .../application_editor/commonlisp.js | 4 +- .../viewerpage/application_editor/css.js | 4 +- .../viewerpage/application_editor/diff.js | 4 +- .../application_editor/dockerfile.js | 4 +- .../viewerpage/application_editor/elm.js | 4 +- .../application_editor/emacs-org.js | 77 ++- .../viewerpage/application_editor/erlang.js | 4 +- .../pages/viewerpage/application_editor/go.js | 4 +- .../application_editor/htmlmixed.js | 4 +- .../viewerpage/application_editor/java.js | 4 +- .../application_editor/javascript.js | 4 +- .../viewerpage/application_editor/jsx.js | 4 +- .../viewerpage/application_editor/lua.js | 4 +- .../viewerpage/application_editor/orgmode.js | 559 +++++++++--------- .../viewerpage/application_editor/perl.js | 4 +- .../viewerpage/application_editor/php.js | 4 +- .../application_editor/properties.js | 4 +- .../viewerpage/application_editor/python.js | 4 +- .../pages/viewerpage/application_editor/r.js | 4 +- .../viewerpage/application_editor/ruby.js | 4 +- .../viewerpage/application_editor/rust.js | 4 +- .../viewerpage/application_editor/sass.js | 4 +- .../viewerpage/application_editor/shell.js | 4 +- .../viewerpage/application_editor/sparql.js | 4 +- .../application_editor/spreadsheet.js | 4 +- .../viewerpage/application_editor/sql.js | 4 +- .../viewerpage/application_editor/stex.js | 4 +- .../viewerpage/application_editor/text.js | 2 +- .../viewerpage/application_editor/xml.js | 4 +- .../application_editor/yaml-frontmatter.js | 4 +- .../viewerpage/application_editor/yaml.js | 4 +- .../viewerpage/application_editor_orgmode.js | 2 +- .../pages/viewerpage/application_form.js | 8 +- .../pages/viewerpage/application_iframe.js | 8 +- .../pages/viewerpage/application_image.js | 11 +- .../pages/viewerpage/application_pdf.js | 20 +- .../pages/viewerpage/application_video.js | 40 +- public/assets/pages/viewerpage/common.js | 6 +- .../assets/pages/viewerpage/common_menubar.js | 2 +- .../assets/pages/viewerpage/common_player.js | 2 +- public/assets/pages/viewerpage/mimetype.js | 2 +- 70 files changed, 536 insertions(+), 547 deletions(-) diff --git a/public/assets/boot/ctrl_boot_frontoffice.js b/public/assets/boot/ctrl_boot_frontoffice.js index 8ca2b088..fa60bf05 100644 --- a/public/assets/boot/ctrl_boot_frontoffice.js +++ b/public/assets/boot/ctrl_boot_frontoffice.js @@ -1,10 +1,8 @@ import rxjs, { ajax } from "../lib/rx.js"; -import { loadJS } from "../helpers/loader.js"; // import { setup_cache } from "../helpers/cache.js"; -import { init as setup_loader } from "../helpers/loader.js"; +import { init as setup_loader, loadJS } from "../helpers/loader.js"; import { report } from "../helpers/log.js"; - export default async function main() { try { await Promise.all([ // procedure with no outside dependencies diff --git a/public/assets/components/breadcrumb.js b/public/assets/components/breadcrumb.js index 98997af6..369e7e18 100644 --- a/public/assets/components/breadcrumb.js +++ b/public/assets/components/breadcrumb.js @@ -1,8 +1,7 @@ -import { animate, slideYOut, slideYIn, slideXIn, opacityOut } from "../lib/animate.js"; +import { animate, slideYOut, slideYIn, opacityOut } from "../lib/animate.js"; import { loadCSS } from "../helpers/loader.js"; class ComponentBreadcrumb extends window.HTMLDivElement { - constructor() { super(); if (new window.URL(location.href).searchParams.get("nav") === "false") { @@ -30,12 +29,12 @@ class ComponentBreadcrumb extends window.HTMLDivElement { switch (name) { case "path": - if (newValue == "") return; + if (newValue === "") return; return this.renderPath({ path: newValue, previous: oldValue || null }); case "indicator": - return this.renderIndicator() + return this.renderIndicator(); } - throw new Error("component::breadcrumb.js unknow attribute name: "+ name) + throw new Error("component::breadcrumb.js unknow attribute name: "+ name); } static get observedAttributes() { @@ -45,7 +44,7 @@ class ComponentBreadcrumb extends window.HTMLDivElement { async renderPath({ path = "", previous }) { path = this.__normalised(path); previous = this.__normalised(previous); - let pathChunks = path.split("/"); + const pathChunks = path.split("/"); // STEP1: leaving animation on elements that will be removed if (previous !== null && previous.indexOf(path) >= 0) { @@ -97,7 +96,7 @@ class ComponentBreadcrumb extends window.HTMLDivElement { ${limitSize(label, true)} `; - return `
${limitSize(label)}
` + return `
${limitSize(label)}
`; })(); return ` @@ -133,17 +132,21 @@ class ComponentBreadcrumb extends window.HTMLDivElement { if (state && this.getAttribute("indicator") !== "false") state = true; const $indicator = this.querySelector(`[data-bind="path"]`) - .lastChild - .querySelector("span"); + .lastChild + .querySelector("span"); if (state) { $indicator.style.opacity = 1; $indicator.innerHTML = `
*
`; - await animate($indicator, { time: 500, keyframes: [ - { transform: "scale(0)", offset: 0 }, - { transform: "scale(1.5)", offset: 0.3 }, - { transform: "scale(1)", offset: 1 }, - ], fill: "none"}); + await animate($indicator, { + time: 500, + keyframes: [ + { transform: "scale(0)", offset: 0 }, + { transform: "scale(1.5)", offset: 0.3 }, + { transform: "scale(1)", offset: 1 }, + ], + fill: "none" + }); } else { $indicator.style.opacity = 0; await animate($indicator, { time: 200, keyframes: opacityOut(), fill: "none" }); diff --git a/public/assets/components/decorator_shell_filemanager.js b/public/assets/components/decorator_shell_filemanager.js index e264923a..bd65f972 100644 --- a/public/assets/components/decorator_shell_filemanager.js +++ b/public/assets/components/decorator_shell_filemanager.js @@ -29,13 +29,13 @@ export default function(ctrl) { ctrl(createRender($main)); ctrlSidebar(createRender(qs($page, `[data-bind="sidebar"]`))); - onDestroy(async () => { + onDestroy(async() => { if ((history.state.previous || "").startsWith("/view/") && location.pathname.startsWith("/files/")) { await animate($main, { time: 100, keyframes: slideYOut(20), fill: "none" }); $main.classList.add("hidden"); } }); - } + }; } async function ctrlSidebar(render) { diff --git a/public/assets/components/dropdown.js b/public/assets/components/dropdown.js index 8a8ad836..5e384987 100644 --- a/public/assets/components/dropdown.js +++ b/public/assets/components/dropdown.js @@ -15,7 +15,7 @@ export default class ComponentDropdown extends HTMLDivElement { } render() { - this.classList.add("component_dropdown", "view", "sort") + this.classList.add("component_dropdown", "view", "sort"); this.appendChild(createFragment(`