Skip to content

Remove name <key> from tmLanguage (fixes #8) #10

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

Merged
merged 1 commit into from
Oct 27, 2015

Conversation

azamat-sharapov
Copy link
Contributor

No description provided.

@yyx990803
Copy link
Member

Hmm, why does this fix #8? Just curious...

@yyx990803
Copy link
Member

Ok, I think this is because giving it that name causes the stylus package to double apply auto completion or something. Just changing the name to a different one fixes it too (and make auto completion work properly!)

Can you confirm and edit the YAML as well? Just change the name to something like source.vue-stylus.embedded.html.

@azamat-sharapov
Copy link
Contributor Author

Hmm, why does this fix #8? Just curious...

@yyx990803 I didn't also understand, I borrowed idea from here 😄

But I think you are right, it may be conflict.
Updated.

yyx990803 added a commit that referenced this pull request Oct 27, 2015
Remove name <key> from tmLanguage (fixes #8)
@yyx990803 yyx990803 merged commit 3b5c418 into vuejs:master Oct 27, 2015
@yyx990803
Copy link
Member

👍

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