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

Allow navigating to non-editable cell by tab when editing=true #3655

Closed
LilTwo opened this issue Feb 8, 2020 · 3 comments
Closed

Allow navigating to non-editable cell by tab when editing=true #3655

LilTwo opened this issue Feb 8, 2020 · 3 comments

Comments

@LilTwo
Copy link

LilTwo commented Feb 8, 2020

[x] bug report => see 'Providing a Reproducible Scenario'
[] feature request => do not use Github for feature requests, see 'Customers of ag-Grid'
[] support request => see 'Requesting Community Support'

Plunkr: https://plnkr.co/edit/UTUPUzogqZ9qPfJ3ouTS?p=preview

In edit mode, tab navigation will skip the cells which are not editable.
It seems that we have no way to modify this behavior.
In the example, I print some message in tabToNextCell, it just get called multiple time until reach the next editable cell.

Current behaviour: Tab navigation skips non-editable cells automatically.
Expected behaviour. tabToNextCell should be able to stop the focus at non-editable cell

@LilTwo LilTwo changed the title Allow to use tab navigation to non-editable cell when editing=true Allow navigating to non-editable cell by tab when editing=true Feb 8, 2020
@shuheb
Copy link

shuheb commented Feb 11, 2020

Hi,

This isn't a bug, this is expected behavior. You can modify the body of the event handler of tabToNextCell to only execute its body when the column is editable as shown below.

tabToNextCell(param) {
if(param.nextCellPosition.column.colDef.editable)
console.log('tab to next cell');

return param.nextCellPosition;

}

ag-grid staff does not provide support through github. If you are a customer please contact us through our official support channel. Note that this issue has been flagged as managed-by-the-community, we also recommend you to check stackoverflow, note that if there is no activity on this issue after a while we will close it.

Thanks

@stale
Copy link

stale bot commented Mar 7, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@sudeep-yadagiri
Copy link

Hi Shuheb, can you please provide the solution.

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

3 participants