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

hiddenColumns option interferes with keyboard movement. #5704

Closed
ahndohun opened this issue Jan 7, 2019 · 4 comments
Closed

hiddenColumns option interferes with keyboard movement. #5704

ahndohun opened this issue Jan 7, 2019 · 4 comments

Comments

@ahndohun
Copy link

ahndohun commented Jan 7, 2019

Description

If hiddenColumns is set on the first or last column of a table row, it will no longer move if the first or last row is reached on the keyboard.

Steps to reproduce

Continue moving left or right of the selected line with the keyboard until you move to the previous or next column.

Demo

Examples that work as expected - https://jsfiddle.net/ak1803bj/1/
An example that fails - https://jsfiddle.net/ak1803bj/2/

Your environment

  • Handsontable version: 6.2.2
@AMBudnik
Copy link
Contributor

AMBudnik commented Jan 7, 2019

That's an interesting case. Thank you for sharing @ahndohun

@AMBudnik
Copy link
Contributor

AMBudnik commented May 8, 2020

8.0.0.-beta1 https://jsfiddle.net/q8aLv0p9/ (broken)
8.0.0-beta2-rev2 https://jsfiddle.net/q8aLv0p9/1/ (solved)

@AMBudnik
Copy link
Contributor

Hi @ahndohun

I have good news! We've just released 8.0.0-beta2 that fixes the following issue. We won't be closing this ticket now. It will be closed after we release the final 8.0.0 version of Handsontable.

Here’s a full list of changes https://github.com/handsontable/handsontable/releases/tag/8.0.0-beta.2
NPM https://www.npmjs.com/package/handsontable/v/8.0.0-beta.2 

All feedback's appreciated. Thank you for your input.

@AMBudnik
Copy link
Contributor

AMBudnik commented Aug 6, 2020

Hi @ahndohun

here's a demo updated to the official (stable) version 8.0.0 https://jsfiddle.net/k7uf2qs0/

I hope that it works for you as well. I would really appreciate all the feedback.

CDN https://cdn.jsdelivr.net/gh/handsontable/handsontable@latest/dist/handsontable.full.css
https://cdn.jsdelivr.net/gh/handsontable/handsontable@latest/dist/handsontable.full.js
NPM npm i handsontable
Release Notes https://github.com/handsontable/handsontable/releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants