Skip to content

migrate comics and PDFs to new BookOsd component#8004

Open
dkanada wants to merge 4 commits into
jellyfin:masterfrom
dkanada:book-osd
Open

migrate comics and PDFs to new BookOsd component#8004
dkanada wants to merge 4 commits into
jellyfin:masterfrom
dkanada:book-osd

Conversation

@dkanada

@dkanada dkanada commented Jun 5, 2026

Copy link
Copy Markdown
Member

Changes

PDF and comic players now use the BookOsd component with their existing subset of UI features.

  • neither player supports fullscreen at this time
  • side arrow navigation was removed from the comics player

Subsequent pull requests will focus on adding fullscreen everywhere, implementing a unified page indicator in React, and extracting book navigation controls into a global helper function for user customization.

Issues

None

Code Assistance

None

@dkanada
dkanada requested a review from a team as a code owner June 5, 2026 14:56
@jellyfin-bot

jellyfin-bot commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Cloudflare Pages deployment

Latest commit 6aa203a
Status ✅ Deployed!
Preview URL https://3463a531.jellyfin-web.pages.dev
Type 🔀 Preview

View build logs

@jellyfin-bot

This comment has been minimized.

@jellyfin-bot jellyfin-bot added merge conflict Conflicts prevent merging stale No activity for an extended length of time labels Jun 5, 2026
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Jun 6, 2026
@thornbill thornbill added enhancement Improve existing functionality or small fixes and removed stale No activity for an extended length of time labels Jun 16, 2026
@thornbill thornbill added this to the v12.0 milestone Jun 16, 2026
@thornbill thornbill moved this from Todo to In Progress in Jellyfin 12.0 Jun 16, 2026

@MBR-0001 MBR-0001 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few observations:
The new OSD for pdf hides/blocks the content at the top and bottom

Screencast_20260627_164313.webm

On the comic player this does not happen

Screencast_20260627_164451.webm

BUT, it seems dragging to change page does not work:

Screencast_20260627_164615.webm

@MBR-0001 MBR-0001 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Previously mentioned issues are resolved, now the pdf viewer is broken on mobile

Screencast_20260712_230255.webm
Image

Comment thread src/plugins/bookPlayer/BookOsd/BookOsd.tsx Outdated
Comment thread src/plugins/bookPlayer/BookOsd/BookOsd.tsx Outdated
Comment thread src/plugins/pdfPlayer/plugin.js Outdated
Comment thread src/plugins/pdfPlayer/plugin.js Fixed
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improve existing functionality or small fixes

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

5 participants