Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#39560 closed defect (bug) (fixed)

4.7 About page causes high CPU usage

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

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 10 years ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
10 years ago

  • Description modified (diff)

@bor0
10 years ago

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


10 years ago

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

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → 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
10 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

#9 @SergeyBiryukov
10 years ago

  • Resolution → fixed
  • Status reopened → 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.