Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#39560 closed defect (bug) (fixed)

4.7 About page causes high CPU usage

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: sergeybiryukov's profile 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 7 years ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
7 years ago

  • Description modified (diff)

@bor0
7 years ago

#2 @bor0
7 years 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.


7 years ago

#4 @jbpaul17
7 years 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
7 years 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
7 years 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
7 years 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
7 years ago

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

#9 @SergeyBiryukov
7 years 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.