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.

Sign up for GitHub

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

Jump to bottom

Columns not adding when pasting data (reopened #1668) #3811

Closed
lil25 opened this issue Oct 13, 2016 · 7 comments
Closed

Columns not adding when pasting data (reopened #1668) #3811

lil25 opened this issue Oct 13, 2016 · 7 comments
Assignees
@lil25 @mrpiotr-dev
Labels
bug Moving Status: Released
Milestone
0.28.4

Comments

@lil25
Copy link

lil25 commented Oct 13, 2016

Hello

I've found issue. Columns are not added on pasting if "manualColumnMove" is true.
Here is demo http://jsfiddle.net/5u5vczcg/5/ try to set "manualColumnMove" to true and paste table data columns is not added. (version 0.28.3)

Thanks in advance.
@ghost
Copy link

ghost commented Oct 13, 2016

Related issue : #3800 ?

@lil25
Copy link
Author

lil25 commented Oct 13, 2016

I'm not sure, but maybe they are connected

@ghost
Copy link

ghost commented Oct 13, 2016

Maybe wait the next release, i see the #3800 is fixed.

@mrpiotr-dev mrpiotr-dev added this to the 0.28.4 milestone Oct 13, 2016
@mrpiotr-dev mrpiotr-dev self-assigned this Oct 13, 2016
@mrpiotr-dev
Copy link
Contributor

mrpiotr-dev commented Oct 13, 2016 •
edited

Thanks for sharing @lil25,
Unfortunately this is another bug. I'll try to fix it before today's release

mrpiotr-dev added a commit that referenced this issue Oct 13, 2016
@mrpiotr-dev
Fix: inserting new columns if needed, when manualColumnMove is enabled.
f158193 
#3811
@mrpiotr-dev mrpiotr-dev assigned jansiegel and unassigned mrpiotr-dev Oct 13, 2016
@krzysztofspilka krzysztofspilka mentioned this issue Oct 13, 2016
Closed
mrpiotr-dev added a commit that referenced this issue Oct 13, 2016
@mrpiotr-dev
Fix: correct column index on unmodifyCol callback. #3811
1a6329b
@jansiegel jansiegel assigned mrpiotr-dev and unassigned jansiegel Oct 13, 2016
@AMBudnik
Copy link
Contributor

Hi @lil25
We already made an update on this issue and hopefully it works fine now. Can you take a minute to check it?

@lil25
Copy link
Author

lil25 commented Oct 13, 2016

Now it is working thanks a lot

@AMBudnik
Copy link
Contributor

Great! Thanks for a prompt reply and wish you a wonderful day.

@wszymanski wszymanski mentioned this issue Oct 23, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lil25 lil25

@mrpiotr-dev mrpiotr-dev

Labels
bug Moving Status: Released
Projects
None yet
Milestone
0.28.4
Development

No branches or pull requests
5 participants
@lil25 @jansiegel @krzysztofspilka @AMBudnik @mrpiotr-dev