iPad Video.js play pause button state

[BPerlakiH]

After closing full screen, the video.js play / pause button is stuck on pause state, it does work, but not updating it’s state.

[kelson42]

I can not reproduce this bug on iOS18. Can we have please a details bug report with a screencast?

[BPerlakiH]

@kelson42 this is the screen recording from iPad iOS 17:

ipad_video_pause_issue.mov

TLDR: the play/pause button works at the beginning, after returning from full-screen, it does work (pauses and plays the video), but the state of the button is not changing.

[BPerlakiH]

These are the 2 states of the button:

[kelson42]

@BPerlakiH Old content, not even distributed anymore: please redo a screencast with the most recent ZIM content. Here again I still miss the clear reproduction steps like requested.

[BPerlakiH]

@kelson42 The same issue with newer content, on iPad OS 17:
https://github.com/user-attachments/assets/393eb204-9a9d-4e48-aeb1-411c5e99b7c7

Reproduction steps:

• start playing a video
• enter full screen
• exit full screen, this pauses the video => expect to see the button:
  
• play video => expect to see the button:
  
• pause video => expect to see the button:
  

[benoit74]

To me this looks a lot like Video.JS bug, to be reported to Video.JS repository.

Do you have the same behavior on Safari or with Kiwix app (if you can force it to load a web URL) on https://videojs.com/city?

[BPerlakiH]

@benoit74
I have tested in Safari and force loaded the url into the Kiwix app, both works fine.
"Our" content is using: Video.js 7.8.1
Whereas the latest release is: 8.17.2

[benoit74]

Thank you! Sorry for this new test (hopefully the last one), but can you please test https://mirror.download.kiwix.org/zim/videos/deus-ex-silicium_fr_all_2024-08.zim ; it is a (very) fresh ZIM with video.js 8.17.3 (which is not even marked latest, but we needed this in youtube scraper to support Chrome 58 and 59, and video.js maintainer told us it was fine and going to be latest soon).

[BPerlakiH]

@benoit74 No worries.
I've downloaded and tested the ZIM, and it works great.
So, it seems that the video.js update alone can solve this issue for us.

[kelson42]

@benoit74 Can we link here to the issues requesting this version of video.js? For each of the scraper relying ont video.js?

[BPerlakiH]

@kelson42 @benoit74 I have also tested the same ZIM file on the iPhone, there's one behaviour that is different there (compared to other ZIM files), is that the video starts playing immediately as the page loads, and switches to full screen, which makes our other issue with the black start screen more prominent:

ZIM_iPhone_auto_fullScreen.mp4

[BPerlakiH]

I am happy to close this issue, as with the latest ZIM file and video js it is resolved.

[benoit74]

@benoit74 Can we link here to the issues requesting this version of video.js? For each of the scraper relying ont video.js?

Sure:

• kolibri: Upgrade video.JS to latest version openzim/kolibri#127
• nautilus: Upgrade video.JS to latest version openzim/nautilus#91
• ted: Upgrade video.JS to latest version openzim/ted#225
• youtube: Upgrade video.js version to fix issue with minification on Chrome 51-59 openzim/youtube#275, fixed with Bump video.js to v8.17.3 openzim/youtube#290