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

[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​

[lessbloat]

23813.diff​ should do it.

[helen]

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.

[lessbloat]

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

[helen]

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​?

[lessbloat]

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

[markjaquith]

In 23807:

Use GET for nav menu selection, and default action.

fixes #23813. props helen.

[johnbillion]

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

[DrewAPicture]

Fixed in [23810]