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

Clicking 3 times in field then leaving causes an exception #32

Closed
blbigelow opened this issue Feb 24, 2015 · 3 comments
Closed

Clicking 3 times in field then leaving causes an exception #32

blbigelow opened this issue Feb 24, 2015 · 3 comments

Comments

@blbigelow
Copy link

If a user clicks at least 3 times in an editable cell field and then navigates away from that field the follow error occurs: Uncaught TypeError: undefined is not a function

@blbigelow blbigelow changed the title Clicking 4 times in field then leaving cause exception Clicking 3 times in field then leaving cause exception Feb 24, 2015
@blbigelow blbigelow changed the title Clicking 3 times in field then leaving cause exception Clicking 3 times in field then leaving causes an exception Feb 24, 2015
@jstclair
Copy link
Collaborator

Thanks for reporting this; I'll look into ASAP

@jstclair
Copy link
Collaborator

By any chance, was this in Safari? I can easily reproduce on Safari (8.0.3)/OSX (10.10.2), but not on Chrome (41.0.2272 64bit). If not, could you let me know the specific browser you're testing on?

I'm pretty sure fixing it for Safari would fix it for your case, but if I can test as close to your environment as possible, that would help confirm it's fixed.

The actual exception I see in Safari, if this looks familiar to you, is:

[Error] TypeError: undefined is not a function (evaluating 'cell._cellReadOnly()')
    cellIsEditable (editableCell.min.js, line 1)
    updateCellValue (editableCell.min.js, line 1)
    endEditingCell (editableCell.min.js, line 1)
    onInputBlur (editableCell.min.js, line 2)
    onInputBlur ([native code], line 0)

@blbigelow
Copy link
Author

Thanks for looking into this so fast. I can repeat the same problem on Chrome(Version 40.0.2214.115 (64-bit)), but it takes 4 clicks.

You are right my original testing was on Safari 8.0.3. The tool has made my life so much easier for my current project so I am very grateful for you work on this.

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

2 participants