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

[Bug]: getSelected row/col start and row/col end incorrect when copying data #10771

Open
mass4441 opened this issue Feb 8, 2024 · 3 comments
Labels

Comments

@mass4441
Copy link

mass4441 commented Feb 8, 2024

Describe the bug

When i call the function getSelected when i select a range of cells i get the correct values no matter if i go from bottom to top or top to bottom as the documentation says :

Start row and start column are the coordinates of the active cell (where the selection was started).

But when i copy a cell (with the little square on the corner of a cell) to a range of cells (or one other cell) then the getSelected always return the range from top to bottom even if i try to copy data from bottom to top.

Video/Screenshots

Enregistrement.de.l.ecran.2024-02-08.a.11.02.06.mov

Provide a link to the demo with the bug reproduction

https://handsontable.com/docs/javascript-data-grid/events-and-hooks/#all-available-handsontable-hooks-example

Handsontable version

14.1.0

Framework version

Vue 3.3.4

Your environment

MacOS + Chrome

@mass4441 mass4441 added the bug label Feb 8, 2024
@AMBudnik
Copy link
Contributor

AMBudnik commented Feb 8, 2024

Hi @mass4441

After we autofill a value, the highlighted cell is the top-left one, that is why we get the coordinates counted this way.

Zrzut ekranu 2024-02-8 o 13 16 29

However, it seems that in Google Sheets the highlight doesn't flip to top-left if previously set for bottom-right, so that might be something to fix, and the hook will follow.

Thank you for sharing I will discuss that with the team.

@mass4441
Copy link
Author

mass4441 commented Feb 8, 2024

Hi @AMBudnik ,

Thanks for your answer, i did not noticed when testing that the highlighted cell was changing so now the result of getSelected make more sense.

+1 for the googlesheet way

Do you know when we can expect a feedback from your discussion with your team ?

@AMBudnik
Copy link
Contributor

AMBudnik commented Feb 8, 2024

Hey @mass4441
I got the confirmation that we'd like to handle the cell highlight like Google and MS. I will update you as soon as we fix it.

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

2 participants