search

modxcms / a11y

MODX Accessibility for the Manager
13 stars 10 forks source link

Open Main menu dropdown outside viewport #30

Closed sottwell closed 8 years ago

sottwell commented 9 years ago

Main menu dropdowns that are longer than the screen height don't scroll, have to zoom out the browser to be able to get to the bottom entries of the admin menu (very small text)

dubrod commented 9 years ago

from what i've been told before we can not change things that are part of this lexicon like the menu. so open to suggestions on how we can alter the menus to make them shorter without sacrificing font size

sottwell commented 9 years ago

Not a matter of making them shorter, it's a matter of making them scrollable.

dubrod commented 9 years ago

can you implement that?

sottwell commented 9 years ago

Me? Wouldn't know where to begin.

theboxer commented 9 years ago

It's not problem just of a11y project, but in each MODX install.

btw. @rthrash not sure why I am assigned, it's html/css issue :)

rthrash commented 9 years ago

I could be wrong, but I think the menus are being built taller than the viewport and due to their positioning are not scrollable. Can add scrolling via CSS, but cannot handle the ExtJS parts that build the menus without taking the viewport height into account.

This definitely applies to all themes, and I've tagged this one port to default as such. It probably makes more sense to fix in 2.3.x and have it be part of this by default.

theboxer commented 9 years ago

Menus are not build via ExtJS, but it header controller, in PHP. So it's pure HTML (at least I hope so).

And yes, it should be fixed in 2.3.x, annoying bug...

rthrash commented 9 years ago

I think it's going to require some JS, because of absolute positioning. Maybe we can try something like: http://stackoverflow.com/questions/11512032/detect-if-dropdown-navigation-would-go-off-screen-and-reposition-it

dubrod commented 8 years ago

2-column-submenu

this solution is ready for some testing