WordPress.org

Make WordPress Core

#23813 closed defect (bug) (fixed)

3.6-Menus: Menu select should use get instead of post

Reported by: DrewAPicture Owned by: markjaquith
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Menus Keywords: 3.6-menus commit
Focuses: Cc:

Description (last modified by DrewAPicture)

@helen commented on ticket:23607:20:

Thinking the select a menu form should be get instead of post.

There's a patch on the tracking ticket: ticket:23607:23607.diff

Attachments (3)

23813.diff (792 bytes) - added by lessbloat 13 months ago.
23813.1.diff (739 bytes) - added by lessbloat 13 months ago.
23813.2.diff (1.8 KB) - added by helen 13 months ago.

Download all attachments as: .zip

Change History (12)

comment:1 DrewAPicture13 months ago

  • Description modified (diff)

lessbloat13 months ago

comment:2 lessbloat13 months ago

23813.diff​ should do it.

comment:3 helen13 months ago

Meant to mention that my reasoning for this is so that the URL changes (bookmarking, copying the link, just looking nice) and so that if the user refreshes for some reason after switching the menu it doesn't prompt them about re-submitting form data.

lessbloat13 months ago

comment:4 lessbloat13 months ago

  • Keywords commit has-patch added

23813.1.diff​ addresses Helen's second concern on ticket:23607:20.

helen13 months ago

comment:5 helen13 months ago

Looks like we use an empty action for forms around the admin rather than just omitting it. Also, there are more forms on the screen. Maybe 23813.2.diff?

comment:6 lessbloat13 months ago

23813.2.diff​ looks good to me. This is ready for commit.

comment:7 markjaquith13 months ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In 23807:

Use GET for nav menu selection, and default action.

fixes #23813. props helen.

comment:8 johnbillion13 months ago

  • Keywords has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

This has had the side effect that after adding a new menu you remain on the new menu screen.

comment:9 DrewAPicture13 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in [23810]

Note: See TracTickets for help on using tickets.