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

No onCellSelected event fired on the first cell (0,0) #1310

Closed
3 tasks done
iamkennytso opened this issue Oct 18, 2018 · 5 comments
Closed
3 tasks done

No onCellSelected event fired on the first cell (0,0) #1310

iamkennytso opened this issue Oct 18, 2018 · 5 comments
Labels

Comments

@iamkennytso
Copy link

iamkennytso commented Oct 18, 2018


Which version of React JS are you using?

☣️ Not officially supported, expect warnings and errors ☣️

  • v16.x.x

Which browser are you using?

  • Chrome

I'm submitting a ...

  • 🐛 Bug Report

Issue Details

  • What the current behavior is
    Clicking on the first cell on initial load won't fire onCellSelected
  • What the desired behvaior is
    Clicking on the first cell would fire onCellSelected

https://i.gyazo.com/9081537ab112f2cb63fae1e84a74ad85.gif

This is reopening #1108
In the thread, having the first cell auto focused seemed to be a problem, so I tried adding a temporary dom element .focus() and then removing the temporary element, but the problem still persists. Not sure if it's a focus issue or not.

@iamkennytso
Copy link
Author

setting enableCellAutoFocus to false does not solve the problem neither

@iamkennytso
Copy link
Author

Is there any update on this?

@iamkennytso
Copy link
Author

The bug is that when loading an editable grid for the first time, clicking on the top left most cell does not trigger a onCellSelected event. This makes it so that custom editors will not load properly. This can be seen in the gif:

https://i.gyazo.com/9081537ab112f2cb63fae1e84a74ad85.gif

In the gif, I'm console logging whenever onCellSelected is triggered. When I click on the top left most cell a few times, nothing console logs. When I click on the second cell, it logs out, and then clicking back on the top left most cell logs as well.

@iamkennytso
Copy link
Author

Can someone throw a bug label on this ticket, please?

@stale
Copy link

stale bot commented Dec 20, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please reopen this if you feel it has been incorrectly closed and we will do our best to look into it. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 20, 2018
@stale stale bot closed this as completed Dec 27, 2018
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

1 participant