fix: hidden content page and hide button improvements#407
Open
4eh5xitv6787h645ebv wants to merge 3 commits inton00bcodr:mainfrom
Open
fix: hidden content page and hide button improvements#4074eh5xitv6787h645ebv wants to merge 3 commits inton00bcodr:mainfrom
4eh5xitv6787h645ebv wants to merge 3 commits inton00bcodr:mainfrom
Conversation
The broad card selector in addLibraryHideButtons() matched chapter/scene cards on movie and TV show detail pages. Added a check to skip cards inside #itemDetailPage that contain .chapterCardImageContainer, matching the exclusion pattern already used by the tag files.
…yfin When a hidden item no longer exists in Jellyfin (e.g. movie was deleted), the card on the Hidden Content page would show no poster and link to a broken detail page. Now when the Jellyfin poster request fails: - Falls back to the TMDB poster image if a posterPath was stored - Switches click handlers to open the Jellyseerr modal instead of navigating to the dead Jellyfin detail page Applied to both movie cards (createItemCard in hidden-content.js) and series group cards (createGroupPoster/createGroupCard in hidden-content-page.js).
…Path When items are hidden via native Jellyfin cards, posterPath is often not stored. If the item is later removed from Jellyfin, the poster fails to load and the card appears blank even when a tmdbId is available. - When Jellyfin poster 404s and posterPath is empty, fetch poster from Jellyseerr API (fetchMovieDetails/fetchTvShowDetails) using the tmdbId - Applied to both createItemCard (hidden-content.js) and createGroupPoster (hidden-content-page.js) - Falls back gracefully: stored posterPath → Jellyseerr API → hide image This PR was developed with AI assistance (Claude). All changes have been reviewed and tested on a live Jellyfin 10.11 instance.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
#itemDetailPage .chapterCardImageContainer).posterPath(common when hidden via native Jellyfin UI), the poster is now fetched from the Jellyseerr API (fetchMovieDetails/fetchTvShowDetails) so removed items still display their poster on the Hidden Content page.Files Changed
js/enhanced/hidden-content.js—addLibraryHideButtons()chapter exclusion,createItemCard()Jellyseerr fallbackjs/enhanced/hidden-content-page.js—createGroupPoster()andcreateGroupCard()Jellyseerr fallbackImplementation Notes
This PR was developed with AI assistance (Claude). All changes have been reviewed and tested on a live Jellyfin 10.11 instance.
Test plan
🤖 Generated with Claude Code