Opened 6 months ago

Last modified 6 months ago

#22682 new defect (bug)

Close Button Breaks in Customizer After Refresh

Reported by: miqrogroove Owned by:
Priority: low Milestone: Future Release
Component: Accessibility Version: 3.5
Severity: minor Keywords:
Cc:

Description

Steps to reproduce (keyboard only):

  1. Open the theme customizer.
  2. Tab past the top buttons.
  3. Press F5.
  4. Now try to use the Close or Cancel button.

Expected result: Should be able to Close or Cancel.

Actual result: The button can still be clicked, but it is broken for keyboard usage.

Found in 3.5-RC2-22961 using Chrome 23 and WinXP.

Attachments (1)

22682.diff (436 bytes) - added by lessbloat 6 months ago.

Download all attachments as: .zip

Change History (9)

Workaround: Press Alt + Left several times to return to the Themes page.

Version 0, edited 6 months ago by miqrogroove (next)

Is this reproducible in 3.4.2?

I assumed this was a new accessibility feature of 3.5?

I couldn't figure out how to get to the customizer by keyboard in 3.4.2. I can action the customizer link, but focus moves to the preview pane and no amount of tabs seems to get me to the customizer controls.

  • Severity changed from normal to minor

Reducing severity as it seems unlikely the user would need to refresh the page.

22682.diff​ forces the redirect - fixing the issue. Before this, if you edited a field and closed without saving, then went back into the customizer, all of your changes were lost anyway - so it doesn't look as though we're losing anything by forcing a redirect all of the time. Koopersmith may think differently. :-)

comment:6 follow-up: ↓ 7   miqrogroove6 months ago

22682.diff​ isn't testing well for me. If I use the keyboard to navigate to the Customizer, hit Enter on the close button, and then try to go back to the Customizer by keyboard, it wont load. I can't reproduce that problem by mouse, and the problem goes away when I remove the patch or disable script debug.

comment:7 in reply to: ↑ 6   lessbloat6 months ago

Replying to miqrogroove:

22682.diff​ isn't testing well for me. If I use the keyboard to navigate to the Customizer, hit Enter on the close button, and then try to go back to the Customizer by keyboard, it wont load. I can't reproduce that problem by mouse, and the problem goes away when I remove the patch or disable script debug.

Yep, good catch. I was able to replicate what you're seeing. This is a weird little bug.

  • Milestone changed from Awaiting Review to Future Release
  • Priority changed from normal to low
Note: See TracTickets for help on using tickets.