Admin menu positioned incorrectly

[phill_brown]

The WordPress menu positions incorrectly in a scenario where it's collapsed, hovered and then expanded again.

1. Select a menu item (easiest to use Settings)
2. Collapse the menu
3. Make the height of the window small enough so the sub-menu is forced to move up the page on hover ie the menu won't be inline with it's original hidden position.
4. Expand the menu

I've attached two screenshots to better explain the issue and a patch that fixes it.

An alternative fix would be to traverse each sub-menu when the collapse link is clicked and remove the top margins.

[nacin]

I think this is no longer reproducible in 3.4.

[phill_brown]

Still seem to be able to produce it on the latest in SVN...

[azaozz]

Following the exact steps from the ticket, can't reproduce it. When the menu is expanded any positioning from any sub-menus is removed: ​http://core.trac.wordpress.org/browser/trunk/wp-admin/js/common.dev.js#L175

[phill_brown]

Ah hang on, my mistake sorry. The error now only occurs when the navigation menu collapses and expands automatically when the browser window resizes. Try this:

1. Select 'settings'

2. Maximise your window with the menu in it's default expanded state

3. Shrink your window so it's small enough to trigger the collapse state.

4. Scroll down the page so the position the settings icon is near the bottom of the window.

5. Hover over the settings icon. The positioning of the window should force the sub-menu heading out of line of the icon ie the menu won't be inline with it's original hidden position. See mouseon.jpg.

6. Maximise the window again so the menu expands. This should produce the broken state similar to that shown in expanded.jpg

Just checked my patch and that still appears to fix the problem. It looks like this ticket was in part duplicated and fixed in ticket 20857.

[SergeyBiryukov]

I can reproduce it with the steps from the latest comment.

Refreshed the patch to reset margin-top, like we do in lines 176 and 213.

[azaozz]

I can reproduce it too. Maybe we don't need both removals of the sub-menu negative margin, but as far as I remember sometimes mouseleave doesn't fire right.

[azaozz]

In [21090]:

Admin menu: reset sub-menus position on mouseleave so they don't break when auto-expanded, props phill_brown SergeyBiryukov, fixes #19562

[nacin]

Reopening for 3.4.1 consideration, or closing as fixed on 3.5.

[nacin]

Per dev chat: good to go for the 3.4 branch.

[azaozz]

In [21097]:

Admin menu: reset sub-menus position on mouseleave so they don't break when auto-expanded, props phill_brown SergeyBiryukov, fixes #19562 for 3.4