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

OnCellValueChanged is not triggered when we click out of the grid #2832

Closed
manjuhavanur01 opened this issue Dec 13, 2018 · 2 comments
Closed

Comments

@manjuhavanur01
Copy link

manjuhavanur01 commented Dec 13, 2018

We are trying to do some cell level validation on "OnCellValueChanged",
Chrome Browser:
The "OnCellValueChanged" will be triggered only when we click on any other cell after editing but doesn't work if the click happens out of the grid.
Edge Browser:
The "OnCellValueChanged" will be triggered only when we click out of the grid but on any other cell after editing.
Please let us know how can we achieve this.

Response Received:
Can you provide a reproducible scenario using one of our examples or your source code in a plunker and we will investigate?
Accepted reproducible scenarios are
A description of the detailed steps to reproduce your behaviour in one of our examples in the docs.
A plunker
If you decide to send us a plunkr, from any example in our website use the plunkr button in there to fork your own code by following the steps below:
Select the framework that is appropriate to you from the drop-down
Open it in plunker. (Use the button plunker in our example)
Add your changes so that the behaviour is reproduced
Save and Freeze the plunker(On the top left corner)
Send us the link to the plunker(You can copy the URL from the browser)
If reporting a bug make sure to state.
Actual behaviour.
Expected behaviour. If possible back this up with our docs/examples if possible

Hi Team,

Please find the following plunkr link with example to reproduce the scenario

https://embed.plnkr.co/PNUZwICHVaAgxIfR/

Actual behavior. [Microsoft Edge]

  1. Double click on any cell
  2. Edit the value and click any other cell (focus out event does not triggers before "oncellValueChanged")
  3. Edit the value and click outside of the Ag-grid (focus out event will be triggered then goes to "oncellValueChanged")

We are calling focus out here because we need to decorate the cell based on the values entered here.

Expected behavior. [Microsoft Edge]

  1. Double click on any cell
  2. Edit the value and click any other cell (focus out event must trigger then it should fire "oncellValueChanged")
  3. Edit the value and click outside of the Ag-grid (focus out event must trigger then go to "oncellValueChanged")
@makinggoodsoftware
Copy link
Contributor

https://www.ag-grid.com/javascript-grid-properties/

stopEditingWhenGridLosesFocus Set this to true to stop cell editing when grid loses focus. The default is the grid stays editing until focus goes onto another cell. For inline (non-popup) editors only.

https://plnkr.co/edit/Pis5A9plovUk33mPuzlr?p=info

Hope this helps

@gauravSng1994
Copy link

The actual prop is stopEditingWhenCellLosesFocus not stopEditingWhenGridLosesFocus

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

No branches or pull requests

3 participants