Fix addGridLayout bug when this plugin is combined with addHiddenColumns plugin

[stephane-klein]

Hi @bryanmylee , here is the path to fix #179 bug.

Best regards,
Stéphane

[bryanmylee]

Hey @stephane-klein, I'm really sorry this took so long.

Unfortunately, I don't think this is the approach that we should be using to solve the issue. With this fix, addGridLayout will always override the table's style without allowing other plugins to intercept or modify the attrs.

Maybe we should use the cell's colspan and colstart prop to determine the position, then add a hook to addHiddenColumns to change the colstart and colspan prop before addGridLayout?

[stephane-klein]

Unfortunately, I don't think this is the approach that we should be using to solve the issue. With this fix, addGridLayout will always override the table's style without allowing other plugins to intercept or modify the attrs.

Ok, I understand.

Maybe we should use the cell's colspan and colstart prop to determine the position, then add a hook to addHiddenColumns to change the colstart and colspan prop before addGridLayout?

I'll try to take the time to come up with a new patch in the next ten days.