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

Redesign library navigation in experimental layout #6075

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thornbill
Copy link
Member

Changes
Still in progress, just pushing up so people can try it out

Issues
N/A

Copy link

sonarcloud bot commented Sep 16, 2024

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit d247907aaeb72c0b74109812e8f130cec36d2c8a
Status ✅ Deployed!
Preview URL https://bc0de370.jellyfin-web.pages.dev
Type 🔀 Preview

@enter-a-random-username
Copy link
Contributor

  • Please remove the dropdown. Going back to home with 2 clicks is awful.
  • Active state
  • Still prefer a sidebar that works similar to mobile with icons, no switch on tablet landscape vs portrait
    image

@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Sep 20, 2024
@sjorge
Copy link

sjorge commented Sep 21, 2024

Having the back button appear also interferes with muscle memory, you're used to hitting spot X and then picking library Y from the drop down. Now you want the other library, spot X might be X+1 which is annoying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge conflict Conflicts prevent merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants