I had a problem with the calls to append newly rendered content to cells that hadn't rendered into the cache. I fixed it by changing the call to use jQuery calls rather than DOM methods.
I think the benefits of this over the one in #194 are:
It is more consistent with the rest of the code that tends to use jQuery methods
The jQuery remove method also removes and cleans up event listeners, which the DOM method does not.
Please note, I find the unit tests fail on my Ubuntu laptop, but it is on the estimate of how wide the fonts are, so I hope on your build systems they will work. All other tests pass.
options.minColumnWidth
- should be enabled empty by default......✓
- should resize columns to match the header content width when using 'headerContent'......X
Expected 244 to be greater than 247. (1)
Expected 223 to be greater than 247. (2)
- should be able to toggle the 'minColumnWidth' property using 'setOptions'......X
Expected 244 to be greater than 247. (1)
Expected 223 to be greater than 247. (2)
I had a problem with the calls to append newly rendered content to cells that hadn't rendered into the cache. I fixed it by changing the call to use jQuery calls rather than DOM methods. I think the benefits of this over the one in #194 are:
Please note, I find the unit tests fail on my Ubuntu laptop, but it is on the estimate of how wide the fonts are, so I hope on your build systems they will work. All other tests pass.