Skip to content

Fix sidebar toggle behavior#482

Open
Klath123 wants to merge 2 commits intodevsecopsmaturitymodel:mainfrom
Klath123:feature-sidebar-toggle
Open

Fix sidebar toggle behavior#482
Klath123 wants to merge 2 commits intodevsecopsmaturitymodel:mainfrom
Klath123:feature-sidebar-toggle

Conversation

@Klath123
Copy link
Contributor

This PR improves the sidebar toggle functionality and layout behavior

Changes

  • Fixed sidebar toggle interaction so the header toggle button hides when the sidebar is open and reappears when closed.
  • Added pointer cursor to the sidebar close button for better UX.

Before

Screen.Recording.2026-02-10.100735.mp4

After

Screen.Recording.2026-02-10.101031.mp4

@wurstbrot wurstbrot requested a review from vbakke February 14, 2026 07:24
@vbakke
Copy link
Collaborator

vbakke commented Feb 15, 2026

Thank you @Klath123! I think you're addition is good .
Of all the issues that has turned stale recently, this is probably the one that got the most attention and gotten a number of different suggestions. 😀

Could you have a look at #353?
Would you mind trying out if we can move the hamburger bar to the left of the logo? That way one can "open" and "close" the menu without moving the mouse pointer? (Like YouTube has)

If you need to experiment with the size of the logo to make it look good, feel free : )

@Klath123
Copy link
Contributor Author

Klath123 commented Feb 15, 2026

Thanks for the review @vbakke , working on what you suggested :)

@Klath123
Copy link
Contributor Author

@vbakke as per the given advice and taking a look at the discussions in #353, I have made a sample wirefame. Please recommend me the changes in this. Once approved, after making certain minor changes will raise the PR :)

Recording.2026-02-16.122034.mp4

@vbakke
Copy link
Collaborator

vbakke commented Feb 16, 2026

This looks good, @Klath123. 🙂👍

Maybe tweak the logo so it fits within the menu width. That's all 🙂
image

@Klath123
Copy link
Contributor Author

sure @vbakke , will make the necessary changes and will make the PR shortly :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants