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

Pass selected node to event #16816

Closed
wants to merge 1 commit into from
Closed

Pass selected node to event #16816

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 9, 2016

Passing just text is good but passing the entire element allow more flexibility, i.e. one can know on what link user clicked

Passing just text is good but passing the entire element allow more flexibility, i.e. one can know on what link user clicked
@TxHawks
Copy link
Contributor

TxHawks commented Jan 9, 2016

@sleewoo -
The selected node is already passed as of #16794, sort of. The original event is passed through the originalEvent property (see here), which allows you to look at originalEvent.target

However, there is a bug, because it is only passed when an item is selected using a mouse, not the keyboard. #16795 fixes this, but have not been accepted yet.

@ghost
Copy link
Author

ghost commented Jan 9, 2016

thanks, somehow missed originalEvent,
hoping your fix will be accepted soon.

@ghost ghost closed this Jan 9, 2016
@ghost ghost deleted the patch-1 branch January 9, 2016 17:51
This pull request was closed.
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

Successfully merging this pull request may close these issues.

2 participants