Skip to content
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

Not working in html (vue), when </script> tag has another tag in line #216

Open
vbgm opened this issue Jan 4, 2018 · 1 comment
Open

Comments

@vbgm
Copy link

vbgm commented Jan 4, 2018

When writing in *.vue format (https://vuejs.org/v2/guide/single-file-components.html), to have a cleaner view, I unite end of previous tag with the <script> tag in one line, for example:

<template>

  ..some HTML

</template><script>

  ..some JS (linting does not work)

</script><style>

  ..some CSS

</style>

The linter-js-standard does not work when there is another tag on the same line as the closing </script> tag. It works only if the <style> tag starts with the new line, so that </script> has no tags after it in line). Would be good if fixed.

@vbgm vbgm changed the title Add support for Vue files (similar to html) Within thml allow process script tags that do not start from new line Jan 4, 2018
@vbgm vbgm changed the title Within thml allow process script tags that do not start from new line Within html allow process script tags that do not start from new line Jan 4, 2018
@vbgm vbgm changed the title Within html allow process script tags that do not start from new line Within html allow to process script within tags that do not start from new line Jan 4, 2018
@vbgm vbgm changed the title Within html allow to process script within tags that do not start from new line Not working in html (vue), when </script> tag has another tag in line Jan 4, 2018
@sonicdoe
Copy link
Collaborator

sonicdoe commented Jan 7, 2018

Thank you, I was able to reproduce this.

Long-term, I’m thinking of integrating eslint-plugin-html. I’ve actually already started working on it, however, it turned out to be more difficult than I initially thought.

Either way, I’m also open to a pull request if anyone has a quick fix for the html-splitter.js.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants