Opened 8 years ago
Last modified 5 years ago
#39239 new enhancement
Persist the video header state between requests
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Future Release | Priority: | normal |
| Severity: | normal | Version: | 4.8 |
| Component: | Themes | Keywords: | has-patch needs-refresh |
| Focuses: | Cc: |
Description
If a visitor pauses the video header, they most likely expect it to remain paused through the duration of their visit. Whether it's paused for accessibility reasons or they just find it distracting, it can be annoying to have it restart on refresh or when returning to the homepage. It can be especially troublesome when the video header is activated for every page on a site.
I've seen it mentioned in a few places, but it would be nice if the paused state was remembered between requests.
Attachments (1)
Change History (7)
This ticket was mentioned in Slack in #core by williampatton. View the logs.
5 years ago
This ticket was mentioned in Slack in #themereview by williampatton. View the logs.
5 years ago
This ticket was mentioned in Slack in #accessibility by williampatton. View the logs.
5 years ago
#6
@
5 years ago
- Keywords has-patch needs-refresh added
- Milestone changed from 5.3 to Future Release
Hi @bradyvercher,
Sorry this took so long to get attention! It appears that the patch is no longer applying and needs a refresh. Are you able to help with that?
I am going to punt this for now, but if the patch can be refreshed before the 5.3 Beta 1 deadline on September 23, it can be moved back into the milestone.
39239.diff persists the video state in local storage if it's available. When the video is paused, the
autoplayproperty is removed or set to0in the YouTube handler to prevent the video from playing automatically.