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

Cells with longer text are misleading copy&paste #5925

Closed
AMBudnik opened this issue Mar 27, 2019 · 3 comments

Comments

Projects
None yet
4 participants
@AMBudnik
Copy link
Contributor

commented Mar 27, 2019

Description

Cells with longer text that are copied from Excel or Google Sheets are misleading copy&paste so me miss the empty cells after paste.

Steps to reproduce

  1. Type The following data into Excel of Google Sheets (B is empty)
    image

  2. Select those 4 cells and paste then into Handsontable
    https://handsontable.com/docs/7.0.0/demo-numeric.html

Result:
The empty cell is skipped and you get 3 cells only

Demo

https://handsontable.com/docs/7.0.0/demo-numeric.html

Your environment

  • Handsontable version: 7.0.0
  • Browser Name and version: Chrome 73
  • Operating System: Windows 10
@swistach

This comment has been minimized.

Copy link
Member

commented Mar 28, 2019

Present since v6.1.0. What is interesting, Excel considers overlapped cells as a colspan

 <tr height=21 style='height:16.0pt'>
<!--StartFragment-->
  <td height=21 colspan=2 width=174 style='height:16.0pt;mso-ignore:colspan;width:130pt'>A very long content</td>
  <td align=right width=87 style='width:65pt'>123</td>
  <td align=right width=87 style='width:65pt'>456</td>
<!--EndFragment-->
 </tr>

with mso-ignore: colspan in inline styles.
Support for merged cells in CopyPaste should resolve this issue.

@serajin

This comment has been minimized.

Copy link

commented May 20, 2019

Adding "word-wrap" option to excel cells will temporary fix this bug, or making sure that no text is overlapping into another cell. While not ideal, it will at least resolve the issue temporarily until this bug is fixed in a newer version of Handsontable.

@AMBudnik

This comment has been minimized.

Copy link
Contributor Author

commented Jun 12, 2019

Thank you for fixing @swistach !
Working version form 7.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.