I found that with the exception of the 'menu' display style, submenu tree items are listed in the order they were created, rather than by weight then title. Here's a patch to fix this problem.

CommentFileSizeAuthor
submenutree_order.patch2.91 KBmark matuschka

Comments

mark matuschka’s picture

mark matuschka commented

By the way, I meant to say that in the patch I indented with Drupal's standard 2 spaces, rather than the tab characters used in the rest of this module. I respectfully suggest the rest of the module be changed to 2-space indent as per the Drupal coding standards.

bengtan’s picture

bengtan commented
Assigned: Unassigned » bengtan
Status: Needs review » Fixed

Thanks for that.

I just assumed that the menu structure would already have the items in order. Evidently not.

I've also done the tab -> space expansion as well. Funny, but my gvim/cream is supposed to do it automatically. Evidently not, again.

I've made a 0.3 release incorporating the changes.

Good to know someone's using the module on a proper dataset. My own site's a bit sparse to use it fully.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.