-
Notifications
You must be signed in to change notification settings - Fork 57
Description
Prerequisites
- Put an X between the brackets on this line if you have done all of the following:
- Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
- Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
- Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
- Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
- Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages
Description
When working on Vue applications, sometimes you need to import .vue files inside of normal .js files, for example, when configuring Vue-Router docs.
I was looking at #128 and the PR that addressed it #142, and noticed that it resolves paths to .vue files inside of other .vue files (via the vue scope), but doesn't add similar configuration to .js files (e.g. to handle the use case above).
Steps to Reproduce
- Inside of a Vue project, try to
importa .vue file inside of a .js file. - Get only .js autocomplete suggestions
Expected behavior: [What you expect to happen]
- Inside of a Vue project, try to
importa .vue file inside of a .js file. - Get only .vue autocomplete suggestions if there are any
Actual behavior: [What actually happens]
- Inside of a Vue project, try to
importa .vue file inside of a .js file. - Get only .js autocomplete suggestions
Reproduces how often: [What percentage of the time does it reproduce?]
Always
Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
Additional Information
On my local copy of default-scopes.js, I added vue to the extensions array for the first JS scope entry and it seems to work. I can submit a PR with the same patch.