Our app just gets upgrade from HT 3 to HT 7.4.2. After upgrade, I noticed the “hidden columns” aren’t working properly. I drilled down this issue to the method “isHidden” in the plugin “hiddenColumns”.
This is the fiddle.
Clicking the button should hide the column “Age”. But it actually hides the column “Name”.
This issue should have existed since HT 7.0.0
According to the doc, the “hideColumns” uses the “visual column index”. What this method does is just set “hiddenColumns” on the plugin instance.
However, by default, the “isHidden” use the physical column index to check whether a column is hidden. (isPhysicalIndex is false if the second parameter is not passed). For this reason, if the visual column index isn’t equal to the physical column index, the “hiddenColumn” plugin will hide incorrect column.
If the intention of the change in the “isHidden” means that “showColumns” and “hideColumns” should start using the physical column index?
If not, then is the change in the “isHidden” correct?