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

Column changes order if i click move column, but will not move it #527

Closed
matkus opened this issue Mar 26, 2013 · 4 comments
Closed

Column changes order if i click move column, but will not move it #527

matkus opened this issue Mar 26, 2013 · 4 comments
Labels

Comments

@matkus
Copy link

matkus commented Mar 26, 2013

Hello

There's small issue with column moving. If you start moving column, but will not move it even for 1 pixel, it will be moved to first column.
For example, if you have 5 columns hover column move place for any non-first, then just click it without moving, it will be moved to first column.

[edit] ok, it's not always moved to first column, i have no idea what decides in that case about order. I guess if i haven't selected order, it is set to something quite random.

@setthase
Copy link

Can you provide jsFiddle for that? Or reproduce on handsontable.com demos?

@slackb0t
Copy link

I reported the same issue..

http://jsfiddle.net/api/post/library/pure/

Scroll to all the way to the right and try to drag and drop the last column anywhere.

This is broken because it is relying on the column header indexes which are incorrect as not all the columns are being rendered.

@matkus
Copy link
Author

matkus commented Mar 27, 2013

In my case, you don't have to scroll or anything.
Open default column move demo (http://handsontable.com/demo/column_resize.html - 2nd demo)
move mouse cursor on column header between column C and D until column move cursor appears.
just click. Don't drag the column, just click an release mouse button. Column will be moved to first place.

@psmolenski
Copy link
Contributor

Fixed in version 0.9.12

jasoncrawford added a commit to jasoncrawford/jquery-handsontable that referenced this issue Aug 13, 2013
handsontable#527

This is a version of the fix in:

https://github.com/psmolenski/jquery-handsontable/commit/983e13a2cb27ece41127e6b76c23be83dac9a53d

... but back-ported to the 0.9.11 branch, basically, because when I
tried to upgrade to 0.9.12 to get this bugfix, it broke column moving in
a different way that I couldn't debug.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants