Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 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
12 years ago

  • Milestone changed from Awaiting Review to 3.8

Moving to 3.8 for investigation.

#2 in reply to: ↑ description @netweb
12 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
12 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
12 years ago

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

#5 @azaozz
12 years ago

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

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

#6 @miqrogroove
12 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
12 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
12 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
12 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
12 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
12 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 12 years ago by azaozz (previous) (diff)
Note: See TracTickets for help on using tickets.