When JavaScript is disabled, jump menus can break in a couple different ways:

  • Query strings and fragments in the jump menu don't work.
  • If the site is installed in a subdirectory, none of the jump menus will work at all.

The attached patch fixes those issues.

Note: I discovered this while working on #1103048: Jump menus don't work when created from the view wizard, and this patch is required to make the tests there work.

Comments

merlinofchaos’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

damien tournoud’s picture

Status: Closed (fixed) » Needs review
StatusFileSize
new1.26 KB

This thing is still really broken. The #submit is not registered in one case, and when the user has not selected any option the submit handler redirects to the front page.

merlinofchaos’s picture

Status: Needs review » Fixed

At first I thought hte submit didn't need to be registered, but if embedded into another form it does. So that's important.

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.