columnSelector with multiple rows and colspan

Question

columnSelector with multiple rows and colspan

ArthurJam opened this issue 8 years ago · 7 comments

Hello,

First, Thx for your version of tablesorter.

I have a problem with columnSelector widget.
When I hide a column in relation with a colspan, the widget works and correclty update the colspan value in the first header row..
Surprisingly, when i hide a simple column, the widget does not mask the column in the first header row.

Demo : https://jsfiddle.net/50v3ppwk/10/

Sorry for my poor english.

Answer 1 · 2016-06-29T21:07:51.000Z

Hi @ArthurJam!

Thanks for reporting this problem. I will look into it soon.

Answer 2 · 2016-06-30T01:23:23.000Z

I've updated the column selector widget in the master branch. Please make sure it is working as expected (demo).

I plan to push a new release soon (meaning, the change will be available in the CDN).

Answer 3 · 2016-06-30T11:26:23.000Z

Hi,
It's working as expected now.
Your responsiveness is impressive, thx.

Answer 4 · 2016-07-11T12:47:25.000Z

A new release is now available!

Answer 5 · 2016-12-13T14:36:20.000Z

I'm reopening this issue as I also have problems when using colspan and rowspan.
See the fiddle here (based on last version available on CDN v.2.28.1).

When I unselect the Portfolio column (the first column) and observe that the column "Portfolio" becomes hidden but also the Type 1 column (the first column of the second header row)
...When I unselect the second column this also hides the second column of the second row....and so on.

Can you plan to add this to a near future release ?
Thanks for this tool, I really like it ;-)

Answer 6 · 2016-12-13T17:56:56.000Z

Hi @ptijul! I've fixed the problem in the master branch. I don't plan on releasing a patch just yet... Once some other minor issues have been resolved, you can expect a new patch....

Here is a demo that points to the columnSelector widget in the master branch.

Answer 7 · 2016-12-14T09:06:55.000Z

Hi Mottie ! Wow ! You're awesome !! So fast.
Thanks for the demo, I need to deliver a new release to our clients by the end of next week so I'll download this current version of the master version temporarily until a new version comes up.

Thanks a lot !!