Chosen breaks tables that are using tabledrag. An attempt was made to fix this in #2097007: Tabledrag doesn't work anymore, but this was not the right approach. It assumes that the name used for identify elements with tabledrag ends in '[weight]' or '[position]', but this is not at all guaranteed. In drupal_add_tabledrag() you can specify any classname to use as an identifier.

For example the Field UI tabledrag handler do not contain 'weight' or 'position', so these have been hardcoded. There are however over a dozen forms with tabledrag left in core that are still exhibiting the problem (e.g. block region list, language overview, book outline, ...) and doing the following grep over the contrib folder in one of my projects also came up with dozens of hits:

$ grep -nir drupal_add_tabledrag | grep -v weight | grep -v position

The proper fix for this is to read out the tabledrag settings and exclude them. These settings are stored in Drupal.settings.tableDrag.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen’s picture

Status: Active » Needs review
FileSize
1.44 KB

Here's a patch. Tested on Chrome Canary (40).