![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#10266 closed enhancement (fixed)
wp_page_menu sort_column value should be "menu_order, post_title"
| Reported by: | nathanrice | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.8.1 |
| Component: | General | Version: | 2.8 |
| Severity: | normal | Keywords: | has-patch tested |
| Cc: | Focuses: |
Description
The default argument array for wp_page_menu will ignore the menu_order sorting method, in favor of sorting by post_title. This seems backwards to me, since many theme authors don't know that a default wp_page_menu call will ignore menu_order by default, rendering the ordering field useless to the user.
I propose making the default value for sort_column => 'menu_order, post_title" for the sake of users.
I've attached a diff with the change made.
Attachments (1)
Change History (5)
#1
@
17 years ago
+1
wp_page_menu() hasn't been around long enough for this to break too many themes.
#2
@
17 years ago
Just got bitten by this one. No matter what I did, the menu_order was ignored. Cheers to Nathan for submitting the diff. Please accept this change.
Note:
See TracTickets
for help on using tickets.
post-template.php diff file to add menu_order as the default sorting method for wp_page_menu