WordPress.org

Make WordPress Core

Opened 9 months ago

Closed 8 months ago

#39560 closed defect (bug) (fixed)

4.7 About page causes high CPU usage

Reported by: SergeyBiryukov Owned by: SergeyBiryukov
Milestone: 4.7.3 Priority: normal
Severity: normal Version: 4.7
Component: Help/About Keywords: has-patch commit fixed-major
Focuses: Cc:

Description (last modified by SergeyBiryukov)

Upon the upgrade to 4.7, some users reported a 100% CPU load after being redirected to the About page, apparently caused by three videos set to autoplay simultaneously. Others made a point that it's also a bad UX for users with slow and/or limited mobile data plans.

We should try to fix it in 4.7.2 or at least avoid doing that for future releases.

Attachments (1)

39560.patch (1.3 KB) - added by bor0 9 months ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
9 months ago

  • Description modified (diff)

@bor0
9 months ago

#2 @bor0
9 months ago

  • Keywords has-patch added

Added patch that removes autoplay and loop for the videos in the about page.

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


8 months ago

#4 @jbpaul17
8 months ago

Patch looks good to me. @SergeyBiryukov or @voldemortensen are either of you good to own this and land it in 4.7.3?

#5 @voldemortensen
8 months ago

  • Keywords commit added

I think the patch looks great, although I'm not sure we need to remove the loop attribute. That's not really the problem. Having said that, I don't care either way if loop is removed. +1 on commit.

#6 in reply to: ↑ description @afercia
8 months ago

Replying to SergeyBiryukov:

We should try to fix it in 4.7.2 or at least avoid doing that for future releases.

I'd totally second this and recommend to always avoid video playing automatically, for accessibility reasons too.

#7 @SergeyBiryukov
8 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 40089:

About page: Remove autoplay and loop attributes on "Theme Starter Content", "Edit Shortcuts", and "Video Headers" videos, originally added as a part of [39512].

For UX and accessibility reasons, it's better to always avoid playing videos automatically.

Props bor0.
Fixes #39560.

#8 @SergeyBiryukov
8 months ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

#9 @SergeyBiryukov
8 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 40090:

About page: Remove autoplay and loop attributes on "Theme Starter Content", "Edit Shortcuts", and "Video Headers" videos, originally added as a part of [39512].

For UX and accessibility reasons, it's better to always avoid playing videos automatically.

Props bor0.
Merges [40089] to the 4.7 branch.
Fixes #39560.

Note: See TracTickets for help on using tickets.