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

Title of HTML page is not updated anymore in 3.0 #298

Closed
kelson42 opened this issue Aug 21, 2024 · 6 comments · Fixed by #300
Closed

Title of HTML page is not updated anymore in 3.0 #298

kelson42 opened this issue Aug 21, 2024 · 6 comments · Fixed by #300
Assignees
Labels
Milestone

Comments

@kelson42
Copy link
Contributor

All the pages have the same title, see:
image

@kelson42 kelson42 added the bug label Aug 21, 2024
@kelson42 kelson42 added this to the 3.1.0 milestone Aug 21, 2024
@dan-niles dan-niles self-assigned this Aug 22, 2024
@dan-niles
Copy link
Collaborator

Should the page title include only the video title (e.g., Les Délires de Maisons de Milliardaires !), or should it also include the ZIM title (e.g., Astrolabe | Les Délires de Maisons de Milliardaires !)?

@kelson42
Copy link
Contributor Author

@dan-niles At this stage, I would do it like it was before, so only the video title. Same for playlist page, etc...

@benoit74
Copy link
Collaborator

benoit74 commented Sep 2, 2024

This is not really a regression but an intended change, discussed in #227

The reason behind it is that we know have one single HTML entry. And the content is dynamically updated with JS. We could hence also update the title dynamically from JS (this is the PR). I tried and failed in kolibri (but maybe my code was different, I do not recall). As far as I remember, some readers where not detecting the dynamic change of document title at all (not really a problem but a waste of code/energy) and some other readers (kiwix-serve if I'm not mistaken) where mixing up titles in browser history: looked like page N had the title of page N-1 in history.

@benoit74 benoit74 closed this as completed Sep 2, 2024
@benoit74 benoit74 reopened this Sep 2, 2024
@benoit74
Copy link
Collaborator

benoit74 commented Sep 2, 2024

I did not intended to close this issue at all, sorry, missclick.

@benoit74
Copy link
Collaborator

benoit74 commented Sep 2, 2024

I will test the ZIM built in the PR and revert.

@benoit74 benoit74 changed the title [REGRESSION] ZIM title metadata is not set properly Title of HTML page is not updated anymore in 3.0 Sep 2, 2024
@benoit74
Copy link
Collaborator

benoit74 commented Sep 2, 2024

As mentioned in #300 (review), I did not experienced again the issues I experienced with Kolibri. I'm probably not as a good frontend dev as @dan-niles ^^ Glad it works as expected.

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

Successfully merging a pull request may close this issue.

3 participants