TwistedInteractive / selectbox_link_field_plus

A better selectbox link field for Symphony
14 stars 9 forks source link

Drag to reorder not committing on save #27

Open nathanhornby opened 11 years ago

nathanhornby commented 11 years ago

I'm able to drag to reorder, however upon saving the order is lost and returns to the default sort order.

Also drag to reorder doesn't seem to work if the 'Hide others' checkbox is selected, often resulting in a browser crash (this one could be due to the number of entries, which is around 1500 at the moment, I'm not sure what the test threshold is for this).

Symphony 2.3.1

iwyg commented 11 years ago

I did a little investigation on the "hide others" issue. SBLP uses jQuery UI sortable. jQuery UI sortable is calculating position for each sortable element while dragging and this is quite expensive for hidden elements. In your case this is the reason why your browser stalls. A possible workaround: use SymphonyOrderable instead of UI Sortable, but this would only work for vertical lists.

I also can confirm that reordering is sometimes lost on saving an entry. So there's two bugs that really need to be fixed.