Skip to content

feat(snippets): add a Dynamic Left Sidebar#791

Merged
theRealPadster merged 21 commits intospicetify:mainfrom
Ender-Wang:main
Jul 5, 2024
Merged

feat(snippets): add a Dynamic Left Sidebar#791
theRealPadster merged 21 commits intospicetify:mainfrom
Ender-Wang:main

Conversation

@Ender-Wang
Copy link
Copy Markdown
Contributor

Dynamic Left Sidebar Description

  • Makes the left sidebar dynamic, so it only shows when you hover over it and pushes the main content to the right.

Motivation

  • Previously, I made a New Hover Panel feat(snippets): add a New Hover Panel #778 that fixed the legacy Hover Panels. But after some time using it, I found it blocks the beginning of a playlist(can be anything) when I hover the left sidebar that activates the hovering, which is the expected hovering behavior, but it's kinda annoying. Then I made this one to be like when you hover the left side edge, the LeftSidebar appears and pushes the content to the right so nothing is blocked.

@Ender-Wang Ender-Wang requested a review from a team as a code owner June 23, 2024 16:29
@Ender-Wang Ender-Wang requested review from rxri and removed request for a team June 23, 2024 16:29
@theRealPadster
Copy link
Copy Markdown
Member

Does this replace the other new hover panel snippet? Should we remove it in the code?

@Ender-Wang
Copy link
Copy Markdown
Contributor Author

Does this replace the other new hover panel snippet? Should we remove it in the code?

Function-wise, it replaces all hovering left sidebar snippets, both can be there in case someone prefers hovering instead of this dynamic one.

@theRealPadster
Copy link
Copy Markdown
Member

Oh, the hover one goes on top of the content vs this moves it to the side?

@Ender-Wang
Copy link
Copy Markdown
Contributor Author

Exactly. This one pushes the main view to the right without blocking anything below it.

@theRealPadster theRealPadster merged commit 28ae0c6 into spicetify:main Jul 5, 2024
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