Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#26476 closed defect (bug) (fixed)

"Log Out" and "Edit My Profile" Links Broken for iPad in RC1

Reported by: miqrogroove's profile miqrogroove Owned by: azaozz's profile azaozz
Milestone: 3.8 Priority: high
Severity: normal Version: 3.8
Component: Administration Keywords: needs-patch dev-feedback
Focuses: Cc:

Description

What I did:

  • Upgraded 3.6 to 3.8-RC1 (r26673)
  • Switched to 2014 theme.
  • Tapped Visit Site on the admin bar.
  • Tapped my gravatar on the front page admin bar.
  • Tapped Edit My Profile

What I expected:

  • Navigate to /wp-admin/profile.php

What happened:

  • Nothing

What I tried:

  • Browsers - Broken on iPad, works on Chrome.
  • Change themes - still broken
  • Reload page - no difference
  • Disabled all plugins - no difference
  • Checked error logs - blank
  • Tapped other links - seem to work
  • Admin screens - works, only broken on front end.

Change History (11)

#1 @nacin
11 years ago

  • Milestone changed from Awaiting Review to 3.8

Moving to 3.8 for investigation.

#2 in reply to: ↑ description @netweb
11 years ago

Cannot reproduce, worksforme

What I did:

  • Upgraded from 3.8RC1 to (r26786)
  • Switched to 2014 theme.
  • Tapped Visit Site on the admin bar.
  • Tapped my gravatar on the front page admin bar.
  • Tapped Edit My Profile

What I expected:

  • Navigate to /wp-admin/profile.php

What happened:

  • Successfully opened /wp-admin/profile.php

Tested on iPad Air iOS 7.04 also checked with half a dozen other themes and everything works as expected.

#3 @azaozz
11 years ago

Confirmed, happens only in landscape mode and only with twenty fourteen. For some reason the toolbar doesn't go into "non-responsive" mode like in the admin.

#4 @miqrogroove
11 years ago

I'm using 2013 also and definitely having problems. Confirmed the problem goes away in portrait rotation.

#5 @azaozz
11 years ago

Actually the toolbar is always in "responsive" mode on iPad on the front, despite that landscape is 1024px.

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

#6 @miqrogroove
11 years ago

Landscape isn't 1024 unless the viewport is set that way... I'm guessing that's why the avatar is a different size in the admin screens than on the front page.

#7 @azaozz
11 years ago

The iPad default viewport is 1024x768px. When set with <meta name="viewport" content="width=device-width,initial-scale=1.0"> it keeps these dimensions, however when initial-scale=1.0 is not there (like in the themes), the width is always 768px.

Not sure if initial-scale=1.0 should be set in Twenty Fourteen (imho it looks quite better in landscape mode).

#8 @netweb
11 years ago

Ahhhh, my bad didn't test both orientations ;)

ThemeLandscapePortrait
Twenty FourteenBrokenWorks
Twenty ThirteenBrokenWorks
Twenty TwelveBrokenWorks
Twenty Eleven (1)BrokenBroken
Twenty Ten (2)WorksWorks

(1) Twenty Eleven has many menu links broken, not just the 'Edit Profile' link
(2) Didn't see that coming, Twenty Ten FTW

#9 @nacin
11 years ago

  • Keywords needs-patch dev-feedback added
  • Priority changed from normal to high

Is this something we can fix in core, outside the theme? Our bundled themes are not the only ones setting these meta tags.

#10 @azaozz
11 years ago

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

In 26803:

Toolbar:

  • Fix submenus in responsive mode (were ignoring the .hover class on a parent).
  • Improve top level items hiding.

Fixes #26476.

#11 @azaozz
11 years ago

That was a strange bug: the links in submenus were firing touchstart and touchend but not click. Seems the CSS :focus on the parent element was being removed too soon causing the submenu to close, so the links could not be "clicked".

Last edited 11 years ago by azaozz (previous) (diff)
Note: See TracTickets for help on using tickets.