-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
add detail page link in context menu #6332
base: master
Are you sure you want to change the base?
add detail page link in context menu #6332
Conversation
Cloudflare Pages deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ESLint doesn't pass. Please fix all ESLint issues.
@dmitrylyzo thanks for your feedback! I've realized I forgot to hide the "Details" link when the user is already on the details page. I tried searching the source for similar
EDIT: i figured it out myself: 2616432 |
Quality Gate passedIssues Measures |
I think it would be better to add an option similar to
and set it to false here:jellyfin-web/src/controllers/itemDetails/index.js Lines 90 to 104 in 6f6d6f2
|
Changes
Currently it is impossible to navigate to the detail page of a playlist item.
this PR adds a detail page link to items context menu:
Issues
no known iussues