The Wayback Machine - https://web.archive.org/web/20200608094729/https://github.com/wenzhixin/bootstrap-table/issues/3462
Skip to content

/ bootstrap-table

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sorting on rowspan tbody breaks table #3462

Open
krassowski opened this issue Nov 22, 2017 · 4 comments
Open

Sorting on rowspan tbody breaks table #3462

krassowski opened this issue Nov 22, 2017 · 4 comments
Assignees
@djhvscf
Labels
confirmed js

Comments

@krassowski
Copy link

@krassowski krassowski commented Nov 22, 2017

When sorting a table with data having a defined rowspan causes the "child" rows within the group of rows to disassociated and break rendered improperly.
jsfiddle. Please see following animation:

ezgif-5-caeb0ca3a9
@krassowski
Copy link
Author

@krassowski krassowski commented Nov 22, 2017

Possibly related to: #3400, #2295 and #823.
@krassowski
Copy link
Author

@krassowski krassowski commented Nov 23, 2017

A less popular drvic10k/bootstrap-sortable manages to handles this case in a clever way, using as many duplicates of rowspan cells as needed to keep a table properly formatted: https://jsfiddle.net/jz5tnq4y/15/.

peek 2017-11-23 15-20
Here is the commit which enabled this, just as reference if @wenzhixin agrees on having this future in and someone was interested in implementing it: drvic10k/bootstrap-sortable@0df36e9. Btw. the code is on MIT license too, so heavy inspiration should not be a problem (if acknowledged).
@vaibhav101
Copy link

@vaibhav101 vaibhav101 commented Dec 5, 2018

Has there been any development on this issue?
@UtechtDustin
Copy link
Collaborator

@UtechtDustin UtechtDustin commented Apr 4, 2019

@vaibhav101
Currently not, but any PR is welcome!
@djhvscf djhvscf self-assigned this Nov 16, 2019
@wenzhixin wenzhixin mentioned this issue Feb 25, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed js
5 participants
@wenzhixin @djhvscf @krassowski @vaibhav101 @UtechtDustin
You can’t perform that action at this time.