thgreasi / ui-sortable-multiselection

Provide multiple element sorting in UI-Sortable
MIT License
30 stars 12 forks source link

Fixes for table td width broken when dragging #32

Closed dashawk closed 7 years ago

dashawk commented 7 years ago

When dragging a table row, the td width is lost.

thgreasi commented 7 years ago

This has been discussed some times in ui-sortable but I know agree that we should be able to provide such a solution in an easier way. I would love it if you could to close this PR an open a respective in UI-Sortable. The approach that I have in mind is to create a new custom option named ui-preserve-size which would control when this code executes. The default 'auto' value would result the code to run for <tr>s but the user will still be able to set it to true or false to fine-tune the directive's behavior. What do you think?

See: https://github.com/angular-ui/ui-sortable/issues/237 See: https://github.com/angular-ui/ui-sortable/issues/192

dashawk commented 7 years ago

I think that is a really good idea, to make the options more dynamic. I will create a PR in ui-sortable for this.