This module absolutely needs pagination! We have over 3000 menu items for a particular site and it just crashes the browser straight up. This module needs pagination similar to views bulk operations (where it shows next 3 rows and previous 3 rows per page).

Comments

donquixote’s picture

donquixote commented

Yes, but..
I agree the form can get quite big, but I believe that pagination does not work for a tree-like structure.
What we need instead is to browse in subfolders.

Btw, which menu is that, where you have 3000 items? "navigation" ?
My own solution for that one is to never touch it with any menu editing form.

But yes, this needs a solution.

terbs’s picture

terbs commented

Unfortunately drupal's core menu editing form is also much too big. There's currently no solution for big menus and it would be perfect if your module addressed this problem.

It's really the javascript dragging that's the killer for big menus. Subfolders won't help much because 1. even those subfolders can get big and 2. It's a lot of extra work to build something like that.

I think the overlapping paging is a perfect solution. Page 1 is 1-103, page 2 is 100-203, page 3 is 200-203, etc. This will get around the problem of dragging items from one page to the next. This will keep most of your code in tact while providing a highly usable solution for large menu trees. You can take a look at views_draggable to see how they do the overlapping paging.

donquixote’s picture

donquixote commented

You can take a look at views_draggable to see how they do the overlapping paging.

Does this work for hierarchical drag/drop?

donquixote’s picture

donquixote commented

btw, another solution to replace drag/drop can be found on
http://drupal.org/project/blockadminlight.
But I don't think this would be a solution for menu editor..

terbs’s picture

terbs commented

Yeah it should work for hierarchy. A user might have to drag a top level branch into 2nd level branch to move it back/forward a page, but at least it can be done! It would be better than the current option, which is nothing.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 6.x-2.1 » 6.x-2.x-dev
Issue summary: View changes

I am closing this feature request, as it's for a not supported project version. If you want to see this feature implemented for a supported project version, please re-open this issue.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Closed (outdated)