exitEditMode getting called twice on move command

[gbedardsice]

When using move commands (tab, arrows, etc.), the exitEditMode method of the editor is being called twice. You can see the error here by simply using a move command.

The reason is, here, when you call enterEditMode, the postRender method of the new cell editor calls focus on the new editor element, which results in a blur event on the old editor. The saveOrCancel method is then called again on the old editor, and then triggers the backgrid:edited event a second time.

You can resolve this simply by moving this line before the if block, to make sure that the old editor is actually removed before entering in the new editor.

[fusioned]

I was about to post the same bug issue as it was replicable in the first example on the backgridjs.com home page, but gbedardsice has this explained more succinctly than I would.

The 1st call to exitEditMode is around 28 calls deep from the initial event, which implies this is a side effect, whilst the 2nd call, which I presume is the intended action, happens 10 calls deep in the stack .
.

[wyuenho]

Thanks for reporting this. Free feel to send me a PR.

[gbedardsice]

Oops, I always forget this.

[wyuenho]

Thank you.