#35649 closed defect (bug) (fixed)
Customizer: spinner animated GIF and high CPU usage
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Customize | Keywords: | has-patch has-screenshots |
Focuses: | ui, performance | Cc: |
Description
As a best practice and as already done for the Theme Installer and the ThickBox modal, see tickets 33322 and 33311, when the spinner animated GIF is used as a background image, the element the background is applied to should be promoted to a new layer and the re-painted area should be reduced to the smallest possible one.
#customize-container
and #customize-preview
should just reuse the same CSS used for the Theme Installer.
Attachments (3)
Change History (12)
#1
@
9 years ago
- Keywords has-patch has-screenshots added
- Owner set to afercia
- Status changed from new to assigned
#4
@
9 years ago
For consistency, I'd propose to make also the Theme Installer preview background #f1f1f1
.
This ticket was mentioned in Slack in #core-customize by afercia. View the logs.
9 years ago
#7
follow-up:
↓ 8
@
9 years ago
Sorry about the late reply, but yes, the #f1f1f1
is definitely better.
#8
in reply to:
↑ 7
@
9 years ago
Replying to celloexpressions:
Sorry about the late reply, but yes, the
#f1f1f1
is definitely better.
Thanks @celloexpressions worth noting in the Theme Installer there's still a white background visible for Internet Explorer and Edge users but that's caused by a different issue, the "white flash while iframe loads", see https://css-tricks.com/prevent-white-flash-iframe/
First pass. In the screenshots below, the re-painted area in Chrome before and after the patch.
@celloexpressions should
#customize-container
keep a white background or this would be an opportunity to change it to#f1f1f1
as the standard background color?