Hi,

The module does not work with popups. The problem is on the line 502 in hierarchical_select.js with the return statement. Removing this sentence is functioning properly.

Regards.

Comments

Title:Not working with popupsCompatibility with pop-ups
Status:Needs review» Fixed

Strange. Yet simple fix. Some testing did not reveal any problems, so I decided it's safe to go. Thanks, committed!

http://drupal.org/cvs?commit=412118
http://drupal.org/cvs?commit=412120

Status:Fixed» Closed (fixed)

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

Status:Closed (fixed)» Active

This commit is having some side effects for me:

I use a two level hierarchy vocabulary. Before this commit the behavior was as follows:
Changing the selection on the first hierarchy level results in a AJAX request to retrieve the second level terms. Changing the second level selection is instantaneous (no AJAX call).

After this patch every selection change leads to an AJAX request, even if it is unnecessary.

Status:Active» Needs work

more meaningful status

Status:Needs work» Closed (fixed)

You're absolutely right, digi24. This was fixed in #1007492: Don't always perform a callback to the server when updating.

Now also fixed for Drupal 5: http://drupal.org/cvs?commit=495496.