Opened 7 years ago
Closed 7 years ago
#40442 closed task (blessed) (fixed)
Widgets: Rename "Custom Menu" widget to "Menu"
Reported by: | melchoyce | Owned by: | westonruter |
---|---|---|---|
Milestone: | 4.9 | Priority: | lowest |
Severity: | normal | Version: | |
Component: | Widgets | Keywords: | has-patch |
Focuses: | Cc: |
Description
The "Custom" part of the widget name is a vestigial label from back when non-custom menus were more of a thing. I think at this point, custom menus are pretty much the default way of making a menu. We could probably drop "Custom" from the widget name.
Attachments (4)
Change History (21)
#3
@
7 years ago
enhancement-40442.diff Applies cleanly and seems to fix that issue.
#4
follow-up:
↓ 5
@
7 years ago
@lukecavanagh Thanks for testing the patch. waiting for the response of @melchoyce :)
#5
in reply to:
↑ 4
@
7 years ago
Replying to gk.loveweb:
Thanks for testing the patch. waiting for the response of @melchoyce :)
Thanks for the patch, @gk.loveweb. :) Hoping to get some more opinions before we make a decision.
#7
@
7 years ago
We should not forget to adjust the string here:
https://core.trac.wordpress.org/browser/trunk/src/wp-admin/nav-menus.php?marks=522#L522
This ticket was mentioned in Slack in #core by gkloveweb. View the logs.
7 years ago
#10
in reply to:
↑ 9
@
7 years ago
Replying to melchoyce:
Would "Navigation Menu" be more clear?
Added patch with the navigation please check it. if any other changes required.
enhancement-40442.1.diff
This ticket was mentioned in Slack in #core by ajayghaghretiya. View the logs.
7 years ago
#13
@
7 years ago
- Keywords needs-refresh added
Getting some errors now when I try to apply the patch:
patching file src/wp-includes/widgets/class-wp-nav-menu-widget.php Hunk #2 FAILED at 17. Hunk #3 succeeded at 59 (offset -2 lines). Hunk #4 succeeded at 80 with fuzz 1 (offset -2 lines). Hunk #5 succeeded at 101 with fuzz 1 (offset -3 lines). 1 out of 5 hunks FAILED -- saving rejects to file src/wp-includes/widgets/class-wp-nav-menu-widget.php.rej
#14
@
7 years ago
- Keywords needs-refresh removed
Submitted patch brings @ajayghaghretiya1's changes current and touches a few more spots where the old label was still in use.
From the perspective of calling
wp_nav_menu()
, there is the custom menu that you specify you want to show and there is the fallback menu. In the widget, there is only a choice of Custom menu. It won't show you the fallback. So in a way, Custom Menu makes sense. The default for the fallback is Pages, which is a different widget.