Shouldn't the admin page be under Site Building > Menus? That's where I initially looked for it.

CommentFileSizeAuthor
#4 menutrails-DRUPAL-6--1.menu_.patch1.54 KBsun
#4 menutrails-DRUPAL-5.menu_.patch1.8 KBsun
#3 menutrails-HEAD.menu_.patch1021 bytessun
#3 menutrails-DRUPAL-6--1.menu_.patch1.01 KBsun
#3 menutrails-DRUPAL-5.menu_.patch1.08 KBsun
#1 menutrails-HEAD.patch914 bytessun
#1 menutrails-DRUPAL-6--1.patch926 bytessun
#1 menutrails-DRUPAL-5.patch995 bytessun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Assigned: Unassigned » sun
Status: Active » Needs review
StatusFileSize
new menutrails-DRUPAL-5.patch995 bytes
new menutrails-DRUPAL-6--1.patch926 bytes
new menutrails-HEAD.patch914 bytes

Agreed.

joshk’s picture

joshk commented

Does this add them as local tasks?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new menutrails-DRUPAL-5.menu_.patch1.08 KB
new menutrails-DRUPAL-6--1.menu_.patch1.01 KB
new menutrails-HEAD.menu_.patch1021 bytes

ok, few changes:

- Path changed to admin/build/menu/trails (no need for duplicating "menu").
- Title changed to "Trails" (ditto).
- Type chenged to local task, appearing right after "Settings".
- Removed description, because local tasks have none.
- Access argument changed to "administer menu", using the same permission as Menu module does.

IMHO, nice clean-up.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new menutrails-DRUPAL-5.menu_.patch1.8 KB
new menutrails-DRUPAL-6--1.menu_.patch1.54 KB

DRUPAL-6--1 synced to HEAD now.

Missed to update README.txt.

akahn’s picture

akahn commented

Note that the description for the 6.x-1.x release still says it is the initial port from 5.x, but it seems that now it is newer than that. Maybe the maintainer can change text?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Committed to HEAD and 6.x.

I'm not interested in committing this to 5.x as well. Hence, marking as fixed. Feel free to re-open this issue.

Status: Fixed » Closed (fixed)

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