[MGNLDEMO-269] Provide fallback images to story background videos Created: 15/Feb/18 Updated: 29/Mar/18 Resolved: 27/Mar/18 |
|
| Status: | Closed |
| Project: | Magnolia Demo Projects |
| Component/s: | None |
| Affects Version/s: | 1.2.2 |
| Fix Version/s: | 1.2.3 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Antonín Juran |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoR: |
Empty
|
||||||||
| Date of First Response: | |||||||||
| Epic Link: | Fix stories demo issues | ||||||||
| Sprint: | Kromeriz 138, Kromeriz 139, Kromeriz 140 | ||||||||
| Story Points: | 3 | ||||||||
| Description |
|
By chance I took a look at the Stories page using Chromium, and since it's missing decoding capabilities compared to Chrome, a Vimeo error message is shown when trying to load videos. (see screenshot) I think we should provide fallback images and load videos asynchronously. This would show something for all browsers, and give the video to browsers that support it. In Firefox for instance the videos are loaded but they're not started, meaning you see a black box during loading and it feels really sluggish, but all you get in the end is a static image. With fallback images you'd get the same result much faster. |
| Comments |
| Comment by Christopher Zimmermann [ 19/Mar/18 ] |
|
Just waiting for 5 seconds has 2 drawbacks that I can see.
I think the root of the problem is that you cannot predict how long it will take before video starts to play. The vimeo API provides a "play" event. I would recommend trying to listen for this and to remove the image once the video has actually started playing. https://github.com/vimeo/player.js/#play |
| Comment by Evzen Fochr [ 26/Mar/18 ] |
|
|
| Comment by Roman Kovařík [ 29/Mar/18 ] |
|
efochr ^Could you explain the picture? |
| Comment by Evzen Fochr [ 29/Mar/18 ] |
|
That picture was added while reopening ticket because of migration diff. |