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

Copy Paste from Excel Multi Rows not working 5.0.0 (Community Edition) #5277

Closed
polatic opened this issue Jul 24, 2018 · 11 comments

Comments

@polatic
Copy link

commented Jul 24, 2018

Description

I create an simple table using latest handsontable community edition : https://cdn.jsdelivr.net/npm/handsontable/dist/handsontable.full.min.js

I've copy multiple rows from excel and paste into new simple table.
Only one records filled with last column contains merging last record + index 1 next record.

This issues occured with version 4.0

I use previous handsontable version 3.0 and it's works well

Steps to reproduce

  1. Create simple table
  2. Copy paste multiple lines from Excel 2016

Demo

No demo

Your environment

  • Handsontable version: 3.0.0
  • Browser Name and version: Chrome
  • Operating System: Mac
@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Jul 24, 2018

Hi @polatic

can you share an example text that does not work?

I copy this

image

into

https://docs.handsontable.com/pro/4.0.0/demo-scrolling.html

and besides a double-quote at the end of the text (which is a bug) everything copies fine.

image

@polatic

This comment has been minimized.

Copy link
Author

commented Jul 24, 2018

Hi @AMBudnik,

Sure, here are some demo 5.0.0:
image

This is using 3.0.0 with same config and excel:

image

@polatic polatic closed this Jul 24, 2018

@polatic polatic reopened this Jul 24, 2018

@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Jul 24, 2018

Could you try to paste the rows into this example https://docs.handsontable.com/pro/5.0.0/demo-scrolling.html?

gif

Have you checked the console for errors?

@polatic

This comment has been minimized.

Copy link
Author

commented Jul 24, 2018

Sure, here is:
handsontable

I checked on console and there no error given

@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Jul 24, 2018

I have been testing it on Windows 10 and now on Mac (High Sierra 10.13.6)

mac

but it just works fine.

@polatic

This comment has been minimized.

Copy link
Author

commented Jul 24, 2018

@AMBudnik anything i can do to share more detail about the bug cause? My specs OS X EL Capitan, Office 15.14, Chrome 67.0. I will test it on Windows and will let you know about the result.

@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Jul 25, 2018

I'm using the same Chrome version on Windows 10 and Mac. Can you save and send the file that you're using as a database?

@kazhe

This comment has been minimized.

Copy link

commented Aug 8, 2018

I can confirm the bug being there in Handsontable 5.0 - and it manifests on Handsontable examples as well. (and the bug is there in both community and Pro versions)
Environment: Mac, latest version of Chrome (68), Excel 15.41 (there is no bug in Excel 16.x).
I added a hook on beforePaste and saw that the data it receives is a one dimensional array instead of a 2d array which should be there (and which you get in Firefox).
Therefore the main requirement for reproduction = Excel v15.x.

@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Aug 13, 2018

We were able to replicate in one device.
Inform forum

@swistach

This comment has been minimized.

Copy link
Member

commented Sep 7, 2018

This issue will be solved within #4931

@AMBudnik

This comment has been minimized.

Copy link
Contributor

commented Oct 18, 2018

I am happy to inform that this issue is solved in v6.1.0 released yesterday

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