I think we also should redesign the browser. And adding an entity to the browser via javascript is not cool. So I think first we should use a table, so you can use tablesort later. And the inserting stuff should happens via ajax. I started with rebuildung the browser. Later I will work on the ajax stuff and the ctools object cache which should we use to store the selected entities.

You can find my last patch here.

Comments

yannickoo’s picture

Please use the patch from comment #3.

The attached patch rewrites the module so that you have a <table> instead of using an unorderered list.

yannickoo’s picture

We also should use drupal's tabledrag instead of jQuery UI Sortable.

yannickoo’s picture

davidseth’s picture

Assigned: Unassigned » davidseth

I tried applying patch at http://drupal.org/node/1516308#comment-5831930 but it won't apply cleanly. Can you re-roll patch using latest changes to the branch? Thanks.

ohthehugemanatee’s picture

I'm not clear - is this patch already committed? Cause I get a table with tablesort just fine.