Default overrideNative to !videojs.browser.IS_SAFARI

[marcelbeumer]

Description

Similar to #168, but instead of defaulting to true, I think overrideNative should default to !videojs.browser.IS_SAFARI, like done here:

http-streaming/index.html

Line 65 in 57a38e9

overrideNative: !videojs.browser.IS_SAFARI

It would allow for a more consistent player experience out of the box. I found out about the overrideNative option after finding out Android/Chrome has issues not showing the loading/waiting indicator, first thinking I stumbled on a major vjs bug.

videojs-http-streaming version

what version of videojs-http-streaming does this occur with?
videojs-http-streaming 1.9.3

videojs version

what version of videojs does this occur with?
video.js 7.5.4

[welcome]

👋 Thanks for opening your first issue here! 👋

If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.
To help make it easier for us to investigate your issue, please follow the contributing guidelines.

[wkoncki]

@marcelbeumer
Just wanted to add the default value should probably be
overrideNative: !videojs.browser.IS_ANY_SAFARI
since iOS/iPadOS devices have good HLS support just like macOS ones. Using IS_SAFARI would make the exception only cover desktop.

[gkatsev]

yeah, we're likely going to use IS_ANY_SAFARI rather than IS_SAFARI as it covers more bases.

[gkatsev]

This has just been merged into master via #843.

[phloxic]

yeah, we're likely going to use IS_ANY_SAFARI rather than IS_SAFARI as it covers more bases.

@gkatsev - so e.g. Chrome on iOS does not have issues with vhs? So far I used !IS_ANY_SAFARI && !IS_IOS ...