[wp-trac] [WordPress Trac] #33228: Customizer initial focus
WordPress Trac
noreply at wordpress.org
Mon Aug 3 17:56:14 UTC 2015
#33228: Customizer initial focus
--------------------------+--------------------------------------------
Reporter: afercia | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Customize | Version: 4.2
Severity: normal | Resolution:
Keywords: has-patch | Focuses: ui, accessibility, javascript
--------------------------+--------------------------------------------
Comment (by afercia):
Refreshed patch fixes also the Theme installer overlay focus, see #27705.
Eventually this patch can be split in two but for now I'd recommend to
keep all together for testing purposes, will ask the accessibility team to
test especially with screen readers running and Safari + VoiceOver.
Three main changes in this patch:
- no more need of new CSS classes, just use the existing ones
- in the Theme installer, added a callback for the iframe `load` event
- the callback is used also to remove the spinner which currently is
always "spinning" in the background and even if it's not visible, is
causing high CPU usage especially on old hardware
Any thoughts, improvements, and feedback welcome.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/33228#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list