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

Activating a FancyTree node causes Backspace to stop working in input fields on IE8 #65

Closed
eerohele opened this issue Sep 30, 2013 · 4 comments
Labels
Milestone

Comments

@eerohele
Copy link

This issue occurs with the newest FancyTree version (as well as older ones I've tried) and does not seem to depend on the jQuery version.

Steps to reproduce

  1. Open this JS Bin link in IE8 (here's a Gist if you don't like JS Bin). Note: This issue does not occur when using a newer IE in IE8 mode — you have to use real IE8.
  2. Type something in the text field and try pressing Backspace. It should work.
  3. Click any of the FancyTree nodes (Node 1, for instance) to activate it.
  4. Try deleting some of the text in the text field by pressing Backspace. It doesn't work.

Something that may help point you in the right direction: I tried commenting out l. 3034 of jquery.fancytree.js and that seems to fix the backspace issue without preventing the node from being activated, but it probably does something else important I didn't get to yet.

The console is of no help, unfortunately, since IE8 doesn't have one.

@mar10
Copy link
Owner

mar10 commented Mar 7, 2014

I tried with native IE 8 and the current code (as refdreenced in you JS Bin sample, and it seemed to work for me.
Could you confirm that the issue can be closed for you?

@eerohele
Copy link
Author

eerohele commented Mar 7, 2014

@mar10: I tried with native IE IE 8.0.7601.17514 and it seems to work great. Thank you!

@mar10 mar10 closed this as completed Mar 7, 2014
@mar10
Copy link
Owner

mar10 commented Mar 7, 2014

thanks for the feedback!

@amit034
Copy link

amit034 commented Mar 12, 2014

confirm. all keyboard navigation works :)

@mar10 mar10 added this to the 2.0.0beta milestone May 11, 2014
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

3 participants