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

Handling of external links seems incoherent #182

Closed
kelson42 opened this issue Apr 27, 2023 · 4 comments
Closed

Handling of external links seems incoherent #182

kelson42 opened this issue Apr 27, 2023 · 4 comments
Milestone

Comments

@kelson42
Copy link
Contributor

kelson42 commented Apr 27, 2023

At https://dev.library.kiwix.org/viewer#mes-quartiers-chinois_fr_all_2023-02/A/mesquartierschinois.wordpress.com/

If I click on the link « deux solitudes », it opens directly the Wikipedia link in a new tab.

But if I click on « plutot chouette » I get

IMG_2565

Not sure what shoukd be the correct behaviour, but this seems incoherent.

@rgaudin
Copy link
Member

rgaudin commented Apr 27, 2023

The WP link is opened in a new tab, as requested by the original website (target _blank). Allowing this, we are deferring rendering to the browser and the rest is not part of the ZIM or replayer. As the URL is an online and working one, it's displayed properly.

The other link on the other hand is a regular link and thus goes through our Service Worker that notices it's not included in the ZIM and displays an appropriate message.

That's for why this happens. I agree user don't mind about this and we should have a coherent behavior.

  • Should we allow new-tab links?
  • Should we improve the not found page design and offer a link (on a new tab) to the online location? Kind of what we have on kiwix-serve

@kelson42 kelson42 added this to the 2.0.0 milestone Nov 4, 2023
@benoit74
Copy link
Collaborator

Issue has changed a bit with Zimit2, where behavior is now coherent so we can probably close this issue.

There is still one weird thing but for me it is a kiwix-serve bug:

Shall we close this ticket and open a kiwix-serve issue?

@rgaudin
Copy link
Member

rgaudin commented May 28, 2024

Yep!

@benoit74
Copy link
Collaborator

openzim/libzim#887 is opened, closing this one.

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

No branches or pull requests

3 participants