Skip to content

Conversation

dominiczaq
Copy link
Contributor

@dominiczaq dominiczaq commented Nov 20, 2016

Changed for mounted and destroyed respectively to fix not working arrow keys navigation.

Change for mounted and destroyed respectively to fix
not working arrow keys navigation
@dominiczaq
Copy link
Contributor Author

Hello,
I've always wondered why vue-devtools doesn't have support for arrow keys navigation in component tree view. I've decided to check it out in code and it looks like it was already there - the only problem was that it was still using removed lifecycle hooks (attached and detached). After changing to mounted and destroyed keys navigation works like charm.

Hope this could be merged and thank you for all the hard work you put into Vue.js!

@yyx990803
Copy link
Member

woah, totally missed this one upgrading to 2.0! thanks!

@yyx990803 yyx990803 merged commit c52227d into vuejs:master Nov 22, 2016
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