fix (sidebar): better config on default

on a mobile like screen, the sidebar wouldn't be hidden entirely, it
would still show the border artifact. We need to make sure the default
is to be of class hidden to prevent such artefact
This commit is contained in:
MickaelK 2024-11-18 17:35:43 +11:00
parent 5181b1b49b
commit 291d989ef9
2 changed files with 1 additions and 2 deletions

View file

@ -13,7 +13,7 @@ export default function(ctrl) {
const urlToPath = (pathname = "") => decodeURIComponent(pathname.split("/").filter((_, i) => i !== 1).join("/"));
const $page = createElement(`
<div class="component_filemanager_shell" style="flex-direction:row">
<div data-bind="sidebar"></div>
<div data-bind="sidebar" class="hidden"></div>
<div style="width:100%;display: flex; flex-direction: column;">
<component-breadcrumb path="${urlToPath(history.state.previous)}"></component-breadcrumb>
<div data-bind="filemanager-children"></div>

View file

@ -42,7 +42,6 @@ export default async function ctrlSidebar(render, nRestart = 0) {
// feature: visibility of the sidebar
const forceRefresh = () => window.dispatchEvent(new Event("resize"));
const isVisible = () => settingsGet({ visible: true }, "sidebar").visible;
if (isVisible() === false) $page.classList.add("hidden");
effect(rxjs.merge(rxjs.fromEvent(window, "keydown")).pipe(
rxjs.filter((e) => e.key === "b" && e.ctrlKey === true),
rxjs.tap(() => {