[v3]: Remove whitespace around inserted links

developit · web-flow · commit 0c22f1f85bd5 · 2018-04-19T19:30:39.000-04:00
diff --git a/src/lib/insert-links-into-head.js b/src/lib/insert-links-into-head.js
@@ -22,12 +22,12 @@ function insertLinksIntoHead({html, links=[]}) {
 
   if (html.includes('</head>')) {
     // If a valid closing </head> is found, insert the new <link>s right before it.
-    return html.replace('</head>', links.join('\n') + '\n</head>');
+    return html.replace('</head>', links.join('') + '</head>');
   }
 
   if (html.includes('<body>')) {
     // If there's a <body> but no <head>, create a <head> containing the <head>.
-    return html.replace('<body>', `<head>\n${links.join('\n')}\n</head>\n<body>`);
+    return html.replace('<body>', `<head>${links.join('')}\n</head><body>`);
   }
 
   throw new Error(`The HTML provided did not contain a </head> or a <body>:\n\n${html}`);

Original file line number	Diff line number	Diff line change
`@@ -22,12 +22,12 @@ function insertLinksIntoHead({html, links=[]}) {`
`22`	`22`
`23`	`23`	`if (html.includes('</head>')) {`
`24`	`24`	`// If a valid closing </head> is found, insert the new <link>s right before it.`
`25`		`- return html.replace('</head>', links.join('\n') + '\n</head>');`
	`25`	`+ return html.replace('</head>', links.join('') + '</head>');`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`if (html.includes('<body>')) {`
`29`	`29`	`// If there's a <body> but no <head>, create a <head> containing the <head>.`
`30`		- return html.replace('<body>', `<head>\n${links.join('\n')}\n</head>\n<body>`);
	`30`	+ return html.replace('<body>', `<head>${links.join('')}\n</head><body>`);
`31`	`31`	`}`
`32`	`32`
`33`	`33`	throw new Error(`The HTML provided did not contain a </head> or a <body>:\n\n${html}`);