WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 4 weeks ago

#39239 new enhancement

Persist the video header state between requests

Reported by: bradyvercher Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.8
Component: Themes Keywords: has-patch needs-refresh
Focuses: Cc:
PR Number:

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)

39239.diff (3.1 KB) - added by bradyvercher 3 years ago.

Download all attachments as: .zip

Change History (7)

@bradyvercher
3 years ago

#1 @bradyvercher
3 years ago

39239.diff persists the video state in local storage if it's available. When the video is paused, the autoplay property is removed or set to 0 in the YouTube handler to prevent the video from playing automatically.

This ticket was mentioned in Slack in #core by williampatton. View the logs.


8 weeks ago

#3 @SergeyBiryukov
7 weeks ago

  • Milestone changed from Awaiting Review to 5.3

This ticket was mentioned in Slack in #themereview by williampatton. View the logs.


7 weeks ago

This ticket was mentioned in Slack in #accessibility by williampatton. View the logs.


7 weeks ago

#6 @desrosj
4 weeks 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.

Note: See TracTickets for help on using tickets.