fix(sidebar): hide sidebar instead of removing it

Previously, the sidebar was fully removed from DOM. This could cause unwanted behavior, such as tests which test its content failing. Now, it is just hidden. This does not affect tests any more.
Merge pull request 'Feat(fonts): Add font' (#162 ) from feature/fonts into main
2026-08-05 19:41:07 +02:00 · 2026-08-04 19:41:49 +02:00
1 changed files with 1 additions and 1 deletions
@@ -36,7 +36,7 @@ const toggleSidebar = function toggleSidebar () {
        <div class="sidebar-controls">
            <SidebarExpandButton @click="toggleSidebar" class="sidebar-expand-button" />
        </div>
-        <div class="sidebar-content" v-if="props.expanded">
+        <div class="sidebar-content" v-show="props.expanded">
            <slot />
        </div>
    </nav>
Author	SHA1	Message	Date
jakob.scheid	1bb70046c3	fix(sidebar): hide sidebar instead of removing it Previously, the sidebar was fully removed from DOM. This could cause unwanted behavior, such as tests which test its content failing. Now, it is just hidden. This does not affect tests any more.	2026-08-05 19:41:07 +02:00
jakob.scheidandGitea	baa924a2b2	Merge pull request 'Feat(fonts): Add font' (#162 ) from feature/fonts into main Deploy on dev / Deploy on dev (push) Successful in 34s Details Reviewed-on: #162 Reviewed-by: Jakob Gregory	2026-08-04 19:41:49 +02:00