After a node has been assigned a menu item, the edit form for that node provides a checkbox labeled, "Check to remove this item from the menu.".

This patch changes the text to, "Delete this menu item."

CommentFileSizeAuthor
delete_menu_item.patch472 bytesRowanw

Comments

stevenpatz’s picture

stevenpatz
he/him
commented
Status: Reviewed & tested by the community » Needs review
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Component: language system » menu system
Category: bug » task

This is a menu issue, and not a bug, but an enhancement request IMHO.

Rowanw’s picture

Rowanw commented

I didn't know what "task" represented, which is why I chose bug. I'll know for next time...

Freso’s picture

Freso commented
Status: Needs review » Reviewed & tested by the community

The patch applies with an offset:

gentoo-vm drupal6 # patch -p0 < delete_menu_item.patch
patching file modules/menu/menu.module
Hunk #1 succeeded at 370 (offset -350 lines).

It doesn't break anything and is a really simple patch. If the wording and the issue in general can be agreed upon, it should be ready to go in, thus marking RTBC - feel free to "degrade" if you don't think the wording is good enough.

JirkaRybka’s picture

JirkaRybka commented

New wording seems better to me, more straightforward, less noise in it. Otherwise means the same, so nothing broken anyway. So I agree.

catch’s picture

catch
he/him
Primary language English
commented

Just a quick +1 for the new wording, makes a lot more sense.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Looks good, agreed. Committed.

Anonymous’s picture

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