Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 21 commits into from
Jul 5, 2024

Conversation

Ender-Wang
Copy link
Sponsor 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
Member

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

@Ender-Wang
Copy link
Sponsor 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
Member

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

@Ender-Wang
Copy link
Sponsor 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
5 checks passed
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.

None yet

3 participants