Fix(sidebar): hide sidebar instead of removing it when it is not expanded #174

Merged
jakob.scheid merged 1 commits from fix/settings-view into main 2026-08-06 12:47:41 +02:00
Owner

This pull requests hides the sidebar instead of removing it from DOM when it is not expanded. Removing it could cause behavior, such as tests which test its content failing. Tests now are no longer affected by the sidebar visibility.

Related issues

Closes #166

This pull requests hides the sidebar instead of removing it from DOM when it is not expanded. Removing it could cause behavior, such as tests which test its content failing. Tests now are no longer affected by the sidebar visibility. # Related issues Closes #166
jakob.scheid added this to the 0.1.0 milestone 2026-08-05 19:45:25 +02:00
jakob.scheid added the
Priority
High
2
Kind/BugArea/Sidebar
labels 2026-08-05 19:45:25 +02:00
jakob.scheid added this to the Project Board project 2026-08-05 19:45:25 +02:00
jakob.scheid moved this to Review in Project Board on 2026-08-05 19:45:29 +02:00
jakob.scheid added 1 commit 2026-08-05 20:46:37 +02:00
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.
jakob.scheid force-pushed fix/settings-view from 1bb70046c3 to 32555d9acb 2026-08-05 20:46:37 +02:00 Compare
jakob.gregory approved these changes 2026-08-06 12:47:05 +02:00
jakob.scheid merged commit b0642cc826 into main 2026-08-06 12:47:41 +02:00
jakob.scheid deleted branch fix/settings-view 2026-08-06 12:47:41 +02:00
jakob.scheid moved this to Done in Project Board on 2026-08-06 12:47:49 +02:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Seekra/frontend#174