Skip to content

Conversation

@HerringtonDarkholme
Copy link
Member

According to the doc, $refs is a dictionary of Vue instance or Element. This pull request reflect this by a union type. So users can declare html elements in $refs.

A test is also added. User defined components may have better knowledge about $refs' data structure. So users can also reify special properties to more specific types.

@ktsn your review is appreciated, thanks.

Copy link
Member

@ktsn ktsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to doc, $refs also can be Array.
So $refs should be Vue | Element | Vue[] | Element[]?

When used on elements/components with v-for, the registered reference will be an Array containing DOM nodes or component instances.

@HerringtonDarkholme
Copy link
Member Author

CI breaks... Any idea on it?

@HerringtonDarkholme
Copy link
Member Author

HerringtonDarkholme commented Sep 26, 2016

It seems like dist files need to be rebuilt.
It seems just time out. Changing waiting time longer will get all tests passed.

@ktsn
Copy link
Member

ktsn commented Sep 26, 2016

Yeah, it happens recently... Don't mind about that.

@yyx990803 yyx990803 merged commit aefb348 into vuejs:next Sep 28, 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.

3 participants