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.

Sign up for GitHub

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

Jump to bottom

Focusing on first tree node when bringing tree into focus with a mouse #2

Closed
zdazzy opened this issue Apr 30, 2013 · 6 comments
Closed

Focusing on first tree node when bringing tree into focus with a mouse #2

zdazzy opened this issue Apr 30, 2013 · 6 comments
Labels
bug
Milestone
v.2.0.0

Comments

@zdazzy
Copy link

zdazzy commented Apr 30, 2013

When clicking on a node (not first one) of an unfocused tree with a mouse the first tree node gets focused, and the node being clicked gets active status. Expected only the node being clicked to receive focus/active status.

Under debugger it seems to work as intended. Seems to be some timing/racing problem.

firefox 20
fancytree 2.0.0pre, commit bee0c40
extensions: themeroller
@mar10 mar10 mentioned this issue May 2, 2013
Closed
@zdazzy
Copy link
Author

zdazzy commented May 5, 2013

any progress on this one?

@mar10 mar10 closed this as completed in 190da28 May 7, 2013
@zdazzy
Copy link
Author

zdazzy commented May 13, 2013

Still doesn't work properly.

@mar10
Copy link
Owner

mar10 commented May 13, 2013

you're right, the patch only seemed to fix it on chrome/safari

@mar10 mar10 reopened this May 13, 2013
@zdazzy
Copy link
Author

zdazzy commented May 13, 2013

Isn't this the line that causes trouble?

@mar10
Copy link
Owner

mar10 commented May 13, 2013

It's required when we use TAB to enter the tree control for the first time. I will look into it, when I am back at my Windows box the next time...

mar10 added a commit that referenced this issue May 25, 2013
@mar10
Worked on the focus issue (needs testing)
9be365c 
Fixed issue #2
@mar10
Copy link
Owner

mar10 commented Mar 18, 2014

This seems fixed, at leas I cannot reproduce it.
Could you confirm?

@mar10 mar10 closed this as completed Mar 18, 2014
mar10 pushed a commit that referenced this issue Apr 29, 2014
Merge pull request #2 from markbernard/enhance_filter
1104c25 
Update sample-ext-filter.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v.2.0.0
Development

No branches or pull requests
2 participants
@mar10 @zdazzy