Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#30703 closed defect (bug) (fixed)

Comments Menu Item Triggers Random Links on iPad

Reported by: miqrogroove's profile miqrogroove Owned by: johnbillion's profile johnbillion
Milestone: 4.1 Priority: normal
Severity: normal Version: 4.1
Component: Administration Keywords: has-patch commit fixed-major
Focuses: accessibility Cc:

Description

This is reproducible on my iPad 3 with iOS 8.1.2:

  1. Open admin in landscape orientation.
  2. Collapse the admin menu.
  3. Tap the thumbtack icon for posts.
  4. Tap the Categories menu item.
  5. Tap the balloon icon for comments.

Expected result: Comments page should open.

Actual result: Users page opens. :(

Also found:

  1. Open admin in landscape orientation.
  2. Collapse the admin menu.
  3. Tap the thumbtack icon for posts.
  4. Tap the Categories menu item.
  5. Tap the Edit link for one of the categories.
  6. Tap the balloon icon for comments.

Expected result: Comments page should open.

Actual result: Themes page opens. :(

Change History (13)

This ticket was mentioned in Slack in #core by miqrogroove. View the logs.


10 years ago

#2 @miqrogroove
10 years ago

I confirmed this is a regression in trunk. In the 4.0.1 site I tested, tapping the comments icon will cause one of the menus to fly out rather than navigating to a different random page.

Last edited 10 years ago by miqrogroove (previous) (diff)

#3 @azaozz
10 years ago

  • Milestone changed from Awaiting Review to 4.1

Confirmed. Happens only on the Posts => Categories and Posts => Tags screens, seems caused by auto-focusing the first input field on page load.

Tapping anywhere on the screen would open the keyboard and auto scroll-into-view. The 'click' event fires after that auto-scroll so the actual click is on a completely different place on the screen. This affects all links/buttons, not just the menu.

This ticket was mentioned in Slack in #accessibility by azaozz. View the logs.


10 years ago

#5 @azaozz
10 years ago

  • Focuses accessibility added

Replacing the (old) JS based autofocus with the HTML5 attribute changes the behavior in iOS Safari a bit but is still quite buggy. Now the onscreen keyboard opens, the page auto-scrolls and then the keyboard closes by itself...

Removing the autofocus for mobile devices for now. Seems we only have that for the Tags, Categories, Edit Tag, Edit Category, and Edit Comment screens. If auto-focusing on page load is better for accessibility, we should use the HTML attribute (except on mobile devices).

Last edited 10 years ago by azaozz (previous) (diff)

#6 @azaozz
10 years ago

In 30842:

Do not autofocus text fields on page load on mobile devices. This is currently broken in many mobile browsers:

  • iOS Safari opens the keyboard and auto-scrolls on the first tap anywhere on the screen triggering the click at an unexpected place. That makes it impossible to follow links or press buttons.
  • Chrome on iOS opens the keyboard on load and may scroll the focused field off screen.
  • The Android 4.4 browser only highlights the field, the user has to tap it to open the keyboard and type.

See #30703.

#7 @nacin
10 years ago

I think we do focusing on setup-config.php and install.php too.

#8 @azaozz
10 years ago

Right. In install.php the focusing is already disabled for mobile devices.

In setup-config.php we focus a readonly textarea that doesn't trigger showing the keyboard. This is still kind of buggy in iOS, may auto-scroll on the first tab. Patching.

Version 0, edited 10 years ago by azaozz (next)

#9 @azaozz
10 years ago

In 30843:

iOS: do not autofocus the readonly textarea with the code for wp-config.php in setup-config.php.
See #30703.

#10 @ocean90
10 years ago

  • Keywords has-patch added

#11 @nacin
10 years ago

  • Keywords commit fixed-major added

#12 @johnbillion
10 years ago

  • Owner set to johnbillion
  • Resolution set to fixed
  • Status changed from new to closed

In 30909:

iOS: do not autofocus the readonly textarea with the code for wp-config.php in setup-config.php.

Merges [30843] to the 4.1 branch.

Fixes #30703.

#13 @johnbillion
10 years ago

[30908] missed this ticket.

Note: See TracTickets for help on using tickets.