#19323 closed defect (bug) (fixed)
Flyout menus don't accomodate large number of submenu pages.
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | low | Milestone: | 3.3 |
| Component: | Administration | Version: | 3.3 |
| Severity: | normal | Keywords: | has-patch needs-testing |
| Cc: |
Description
If a large number of submenu items are added to WordPress 3.3's new flyout menus, the flyout doesn't scroll, making some items inaccessible.
The attached sample plugin adds 50 items to illustrate the problem. In practice, we (Oomph, Inc.) ran into this problem with a necessarily-large menu in a client's theme.
Attachments (3)
Change History (11)
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 3.3
- Priority changed from normal to low
- Component changed from Menus to Administration
"Menus" is for Appearance > Menus. Yeah, it's confusing.
devinreams — 18 months ago
Don't offset the margin for really tall sub-menus (which ends up being negative) if taller than the window
comment:3
devinreams — 18 months ago
- Keywords has-patch needs-testing added; needs-patch removed
What I'd like to see for 3.3 is for the flyouts to not hide under the admin bar when the submenus reproduce like rabbits.
What if we tell the JS not to add the margin-top in the case it's taller than the window? I'm not sure my approach is sound here. Testing needed.
devinreams — 18 months ago
Actually, let's slide the flyout up a bit. This way we can see it and it doesn't break the page if it has to go too low (e.g.: add options-general.php to the test plugin).
- Milestone changed from 3.3 to Future Release
When you click on the top menu, the submenus are shown underneath, scrollable. Seems this is just a rare edge case that can wait for 3.4.
- Milestone changed from Future Release to 3.3
Marked #19362 as a duplicate. There, there is a screenshot that's certainly within the realm of possibility.
Let's review and clean up devinreams' approaches and put this in for 3.3. At least make it not look so obviously broken.
- Owner set to azaozz
- Status changed from new to assigned
azaozz is going to take a look.
Desired 3.3 behavior: The top of the flyout is not hidden under the admin bar.
- Resolution set to fixed
- Status changed from assigned to closed
In [19487]:
comment:8
DrewAPicture — 18 months ago
Looks like something got a little muddled in r19487?
Good thing is, they're not completely inaccessible, as you can click the top-level item. Having 30 pages under Settings is not entirely crazy, so I understand this.
What I'd like to see for 3.3 is for the flyouts to not hide under the admin bar when the submenus reproduce like rabbits. At least then it'll feel usable. Full accessibility can be punted. It can be noted that other workarounds include A) tabbing through o and B) scrolling with a mouse wheel or trackpad.