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

fix: ignore vite-plugin-inspect generated data attributes #298

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

zhuoqi-chen
Copy link
Contributor

@zhuoqi-chen zhuoqi-chen commented Feb 15, 2023

When I integrate @nuxt/devtools or vite-plugin-inspect in nuxt3 project, it reports an error.

nuxt.config.ts config

  modules: [
    [
      '@nuxtjs/html-validator',
      {
        usePrettier: true,
      },
    ],
    '@nuxt/devtools',
  ],

The code generated by vite-plugin-inspect

image

The error message

error: failed to tokenize "=\"layouts/...", expected attribute, ">" or "/>" (parser-error) at inline:73:28:
  71 |     <div id="__nuxt">
  72 |       <!--[-->
> 73 |       <div class="relative"="layouts/default.vue:2:3">
     |                            ^
  74 |         <div
  75 |           class="mx-auto flex h-16 max-w-7xl items-center justify-between px-4 md:px-6 xl:px-16"
  76 |          ="components/TheHeader.vue:2:3"

@netlify
Copy link

netlify bot commented Feb 15, 2023

Deploy Preview for html-validator canceled.

Name Link
🔨 Latest commit 1cd187f
🔍 Latest deploy log https://app.netlify.com/sites/html-validator/deploys/63ec765a6f5bb1000898f9de

Copy link
Collaborator

@danielroe danielroe left a comment

Choose a reason for hiding this comment

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

thanks!

@danielroe danielroe changed the title fix: ignores vite-plugin-inspect generated data attributes fix: ignore vite-plugin-inspect generated data attributes Feb 17, 2023
@danielroe danielroe merged commit 9e8dcd9 into nuxt-modules:main Feb 17, 2023
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