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.

Support from Acquia helps fund testing for Drupal Acquia logo

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
FileSize
1.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.