feat: use modulepreload links

5 years ago · 0025af12f4
parent f3ed7d560e
commit 0025af12f4
2 changed files with 28 additions and 31 deletions
--- a/src/build/render.ts
+++ b/src/build/render.ts
@ -34,17 +34,19 @@ export async function renderPage(
  const pageData = JSON.parse(__pageData)

  const assetPath = `${config.site.base}${ASSETS_DIR}`
-  const renderScript = (file: string) => {
-    return `<script type="module" async src="${assetPath}${file}"></script>`
-  }

-  // resolve imports for index.js + page.md.js and inject script tags for
-  // them as well so we fetch everything as early as possible without having
-  // to wait for entry chunks to parse
-  const pageImports = resolvePageImports(config, page, result)
-  const pageImportScripts = pageImports.length
-    ? pageImports.map((i) => renderScript(i)).join('\n') + `\n    `
-    : ``
+  const preloadLinks = [
+    // resolve imports for index.js + page.md.js and inject script tags for
+    // them as well so we fetch everything as early as possible without having
+    // to wait for entry chunks to parse
+    ...resolvePageImports(config, page, result),
+    pageJsFileName,
+    'index.js'
+  ]
+    .map((file) => {
+      return `<link rel="modulepreload" href="${assetPath}${file}">`
+    })
+    .join('\n    ')

  const html = `
 <html lang="en-US">
@ -53,14 +55,14 @@ export async function renderPage(
    config.site.title
  }</title>
    <meta name="description" content="${config.site.description}">
-    <link rel="stylesheet" href="${assetPath}style.css">${renderHead(
-    config.site.head
-  )}${renderHead(pageData.frontmatter.head)}
+    <link rel="stylesheet" href="${assetPath}style.css">
+    ${preloadLinks}
+    ${renderHead(config.site.head)}
+    ${renderHead(pageData.frontmatter.head)}
  </head>
  <body>
    <div id="app">${content}</div>
-    ${pageImportScripts}${renderScript(pageJsFileName)}
-    ${renderScript(`index.js`)}
+    <script type="module" async src="${assetPath}index.js"></script>
  </body>
 </html>`.trim()
  const htmlFileName = path.join(config.outDir, page.replace(/\.md$/, '.html'))
@ -89,19 +91,16 @@ function renderHead(head: HeadConfig[]) {
  if (!head || !head.length) {
    return ''
  }
-  return (
-    `\n    ` +
-    head
-      .map(([tag, attrs = {}, innerHTML = '']) => {
-        const openTag = `<${tag}${renderAttrs(attrs)}>`
-        if (tag !== 'link' && tag !== 'meta') {
-          return `${openTag}${innerHTML}</${tag}>`
-        } else {
-          return openTag
-        }
-      })
-      .join('\n    ')
-  )
+  return head
+    .map(([tag, attrs = {}, innerHTML = '']) => {
+      const openTag = `<${tag}${renderAttrs(attrs)}>`
+      if (tag !== 'link' && tag !== 'meta') {
+        return `${openTag}${innerHTML}</${tag}>`
+      } else {
+        return openTag
+      }
+    })
+    .join('\n    ')
 }

 function renderAttrs(attrs: Record<string, string>): string {
--- a/src/markdownToVue.ts
+++ b/src/markdownToVue.ts
@ -50,10 +50,8 @@ export function createMarkdownToVueRenderFn(
      ? injectPageData(data.hoistedTags || [], pageData)
      : data.hoistedTags || []

-    // double wrapping since tempalte root node is never hoisted or turned into
-    // a static node.
    const vueSrc =
-      `<template><div><div class="vitepress-content">${html}</div></div></template>\n` +
+      `<template><div class="vitepress-content">${html}</div></template>\n` +
      additionalBlocks.join('\n')
    debug(`[render] ${file} in ${Date.now() - start}ms.`)