(I wasn't sure if this was a code, user interface or miscellaneous component!)

Whenever a page is edited, the primary link menu title attribute gets reset (nulled) when the edits are saved. Disabling the Menu Attributes Module stops this behavior.

After a page edit, you must manually reset the menu description in the Site building | menus | primary links page.

It would be nice if the menu description (title attribute was left alone by the Menu Attributes Module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Schoonzie’s picture

Assigned: Unassigned » Schoonzie

Thanks for pointing this out. I will have a look for the cause and release an update soon.

itsnotme’s picture

Any update in the making...? *inquiring mind*

obsidiandesign’s picture

The issue is that the Menu Attributes module overwrites the $form['menu']['options'] array, which contains the description (['options']['attributes']['title']). I have a pseudo-patch, but I'm trying to follow some of the logic that the module is using and how FAPI handles it. I'll post later, it's too late here to finish it.

Bryan O'Shea
Obsidian Design

obsidiandesign’s picture

Status: Active » Needs work
FileSize
4.69 KB

Attaching patch, may need work, but appears to make things work correctly on my end.

Bryan O'Shea
Obsidian Design

builderShawn’s picture

FileSize
11.75 KB

Possible fix would be to add a Title/Description edit area to the module. This would allow the user to change the title/description when creating a page or editing the page, instead of having to edit this area from the menu.

Dave Reid’s picture

Status: Needs work » Fixed

This should be working as expected now with the backported code form D7.

Status: Fixed » Closed (fixed)

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

dgastudio’s picture

Version: 6.x-1.4 » 7.x-1.x-dev
Priority: Normal » Major
Status: Closed (fixed) » Active

problem still active in d7.

joelpittet’s picture

Issue summary: View changes
Status: Active » Fixed

Closing this as fixed as it's been a long time and nobody else has reported this problem.

@kervi if this issue still exists in 7.x dev please re-open. Just grooming the queue.

Status: Fixed » Closed (fixed)

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