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.
Comment | File | Size | Author |
---|---|---|---|
#1 | 2369721-1-chosen-tabledrag_broken.patch | 1.44 KB | pfrenssen |
Comments
Comment #1
pfrenssenHere's a patch. Tested on Chrome Canary (40).