The admin/build/menu-customize/ pages are slow to load and sometimes white screen. The majority of the work involved in loading the menus are the large select tags (at least 101 elements per menu item) and the javascript to make them draggable. The select tags can be replaced with textfields with no change in functionality for javascript users and minimal change in functionality for non-javascript users (a textfield rather than a bulky 101+ item selection.)

CommentFileSizeAuthor
#2 menu.patch1.08 KBjuli
menu.patch821 bytesjuli
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vosechu’s picture

Too much whitespace, needs comments. Have you tried this in 7?

Comment guidelines: http://drupal.org/coding-standards#comment

juli’s picture

FileSize
1.08 KB

Cleaned up patch and added comments.

In Drupal 7, the form processing specifically looks for and uses the options, so unfortunately a more complicated solution is needed. I haven't yet worked on a 7 site with large menus, so I can't say whether such a solution is necessary.

vosechu’s picture

I think we're going to have to test it on a 7 box in order to get any love at this point. Actually, we may still not get any love for a while but we can still push people on it.

For reference the site we were testing on had 1500 menu items. I suspect that anything over 500 will freeze IE and anything over about 1k will probably timeout most other browsers no matter how fast their rendering engine is.

Status: Patch (to be ported) » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.