Skip to content

feat: find correct child components' definition inside template region #2075

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

Closed
wants to merge 4 commits into from

Conversation

hikerpig
Copy link
Contributor

NOTE: This is a copy of PR 1760, sorry that I republish original branch fixing conflicts but got messed up by github's workflow, had to create a new PR here.

This PR will add support for #1712.

I notice that there is already vueFileInfo.componentInfo.childComponents provided by the implementation of Vue interpolation. But during definition-finding for html, a little case issue causes this functionality to fail, typically when component filenames adopt a PascalCase convention.

@hikerpig hikerpig marked this pull request as draft July 31, 2020 03:17
Copy link
Member

@octref octref left a comment

Choose a reason for hiding this comment

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

Let's fix CI test first

@hikerpig hikerpig marked this pull request as ready for review July 31, 2020 12:40
Copy link
Contributor Author

@hikerpig hikerpig left a comment

Choose a reason for hiding this comment

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

Thanks

@hikerpig
Copy link
Contributor Author

Sorry for the LSP tests paths, long time after my first commit. I've merged the master branch and fixed some conflicts.
If everything is good, I wish to rebase and squash these commits before submit 😃

@octref
Copy link
Member

octref commented Aug 3, 2020

Seems covered by #2096, which I merged a bit earlier. I didn't see this PR, sorry.
#2096 covers kebab case as well in case you have a PascalCase component.

@octref octref closed this Aug 3, 2020
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