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

Links in episode "Overview" (<plot>) don't work as expected #984

Open
Zauberfisch opened this issue Mar 4, 2023 · 0 comments · May be fixed by #1395
Open

Links in episode "Overview" (<plot>) don't work as expected #984

Zauberfisch opened this issue Mar 4, 2023 · 0 comments · May be fixed by #1395
Labels
bug Something isn't working confirmed Confirmed bug reports that shouldn't go stale

Comments

@Zauberfisch
Copy link

Zauberfisch commented Mar 4, 2023

Describe the bug

An episode "Overview" (<plot>) can contain html to describe the content, including links (<a>).
This works well in the browser version of jellyfin, however the android app will treat them as app-internal-links.

current behaviour

Links in <plot> open as part of the app.
If you click on a link, it will open that link inside the jellyfin app as if it was part of the app.

when a link is clicked:

  • there is no way to go back (other than closing the app and restarting)
  • no browser bar is shown
  • no way to open a link externally in another app (browser)

expected behaviour

links in <plot> should be opened externally (open in the default android app that handles that type of link. For example open normal website links with the browser, open youtube links with the youtube app, ...)

My usecase

I am downloading videos and podcasts from websites and create .nfo files for them. In the <plot> I add the original link. Sometimes I want to open them to view comments or lookup other details.

How to reproduce

  1. edit the overview of an episode to contain an html link (this can be done via UI or <plot> in .nfo**)
  2. view the episode in the android app. it will show the plot and show the link as a blue clickable link
  3. click on the link

** if you edit the .nfo file, you have to escape the html. so for example:
<plot>&lt;a href="https://github.com/jellyfin/jellyfin-android"&gt;my test link&lt;a&gt;</plot>

Logs

No response

Application version

2.4.4

Where did you install the app from?

F-Droid

Device information

Sony Experia XA2 Ultra

Android version

LineageOS 11

Jellyfin server version

10.8.9

@Zauberfisch Zauberfisch added the bug Something isn't working label Mar 4, 2023
@nielsvanvelzen nielsvanvelzen added the confirmed Confirmed bug reports that shouldn't go stale label Mar 4, 2023
@AllegraCodes AllegraCodes linked a pull request May 22, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed Confirmed bug reports that shouldn't go stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants