Skip to content
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

[8.0.0+] toHTML() returns an error if only colHeaders or rowHeaders are on #8041

Closed
AMBudnik opened this issue May 17, 2021 · 2 comments · Fixed by #8323
Closed

[8.0.0+] toHTML() returns an error if only colHeaders or rowHeaders are on #8041

AMBudnik opened this issue May 17, 2021 · 2 comments · Fixed by #8323
Assignees
Labels
bug Regression Issues that were created while adding new changes to the source code Status: Released

Comments

@AMBudnik
Copy link
Contributor

Description

The toHTML() returns an error if colHeaders are on in versions 8+

Steps to reproduce

  1. Go to https://jsfiddle.net/handsoncode/bqepxasw/ (7.4.2.)
  2. Log result
    image
  3. Go to https://jsfiddle.net/handsoncode/3mg8rsxc/ (8.0.0)
  4. Log result
    image

Details

Demo

https://jsfiddle.net/handsoncode/1efLzgu7/ 7.4.2 - last working
https://jsfiddle.net/handsoncode/3mg8rsxc/ 8.0.0 - first broken
https://jsfiddle.net/handsoncode/6ua5efqr/ - (current) 8.4.0 - still broken

Your environment

  • Handsontable version: 8+
  • Browser Name and version: Chrome 89
  • Operating System: Windows 10
@AMBudnik AMBudnik added bug Regression Issues that were created while adding new changes to the source code labels May 17, 2021
@mrpiotr-dev mrpiotr-dev linked a pull request Jun 28, 2021 that will close this issue
2 tasks
@mrpiotr-dev mrpiotr-dev self-assigned this Jun 28, 2021
@aninde
Copy link
Contributor

aninde commented Jul 20, 2021

This issue will be fixed by v9.0.2
Demo https://jsfiddle.net/zd2cL7gh/

Result from v9.0.2:
<table><thead><tr><th>A</th><th>B</th><th>C</th></tr></thead><tbody><tr><td >3</td><td >5</td><td >6</td></tr><tr><td >6</td><td >7</td><td >9</td></tr></tbody></table>

@mrpiotr-dev
Copy link
Contributor

I'm happy to announce we just released v9.0.2 with the fix to this bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Regression Issues that were created while adding new changes to the source code Status: Released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants