Skip to content
This repository has been archived by the owner on May 3, 2021. It is now read-only.

Video Playback on iOS #273

Open
morbificagent opened this issue Jul 13, 2019 · 4 comments
Open

Video Playback on iOS #273

morbificagent opened this issue Jul 13, 2019 · 4 comments

Comments

@morbificagent
Copy link

Detailed description of the problem

When you are clicking on a Video it opens it in the "bigger" (Fullscreen View) but it does not start playing.

There seems to be some invisible buttons (sometimes as gray areas on the top right and left of the Video) and if you are trying to tab on it it sometimes starts playing.

All in all starting a video on iOS isn't really possible.

Steps to reproduce the issue

Output of the diagnostics (Settings => Diagnostics)

Browser and system

@kamil4
Copy link
Contributor

kamil4 commented Jul 13, 2019

Is your display wide enough for Lychee to display the previous/next arrows on the sides? We don't include them until the viewport width is over 640 CSS pixels. Those arrows have wrapper areas covering 15% of viewport width on each side and, until recently, they would cover the videos, which could make it impossible to select "Play".

The bug was fixed 3 days ago in commit LycheeOrg/Lychee-front@ab80532 so it's not included in any release. Worse, it's not clear if we will make any more Lychee v3 release as we are preparing to release v4, which can be found here: https://github.com/LycheeOrg/Lychee-Laravel. I encourage you to give that new version a try and see if it fixes your issue. Note that the installation process is slightly more complex than with v3.

@morbificagent
Copy link
Author

morbificagent commented Jul 14, 2019 via email

@d7415
Copy link
Contributor

d7415 commented Jul 14, 2019

Is it possible to "upgrade" to v4 at the moment?

This should still work (we'll have to test it more before the v4 release, but it has worked and I don't think we've broken anything...

Worse, it's not clear if we will make any more Lychee v3 release as we are preparing to release v4

I'd think we'd release at least one more v3 if anything (like that) has changed, before breaking support in Lychee-front, but yeah, no guarantees at this point.

@kamil4
Copy link
Contributor

kamil4 commented Jul 15, 2019

As @d7415 said, upgrading is supported. I'd suggest though that you first try a "clean" install just to test whether the issue you reported has been fixed. You'll just need to clone the Lychee-Laravel repo. It includes a precompiled front end with the video fixes I mentioned.

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

No branches or pull requests

3 participants