change default behaviour to keep default values of form elements (#1249)

parcel-bundler · Apr 29, 2018 · f75cc44 · f75cc44
1 parent 8cfe0e2
commit f75cc44
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 0 deletions.
diff --git a/packages/core/parcel-bundler/src/transforms/htmlnano.js b/packages/core/parcel-bundler/src/transforms/htmlnano.js
@@ -7,6 +7,7 @@ module.exports = async function(asset) {
   const htmlNanoConfig = asset.package.htmlnano ||
     (await asset.getConfig(['.htmlnanorc', '.htmlnanorc.js'])) || {
       collapseWhitespace: 'conservative',
+      removeRedundantAttributes: false,
       minifyCss: {
         safe: true
       }

diff --git a/packages/core/parcel-bundler/test/html.js b/packages/core/parcel-bundler/test/html.js
@@ -290,6 +290,16 @@ describe('html', function() {
     );
   });
 
+  it('should not minify default values inside HTML in production mode', async function() {
+    await bundle(__dirname + '/integration/htmlnano-defaults-form/index.html', {
+      production: true
+    });
+
+    let html = fs.readFileSync(__dirname + '/dist/index.html', 'utf8');
+    assert(html.includes('<input type="text">'));
+    assert(!html.includes('\n'));
+  });
+
   it('should not prepend the public path to assets with remote URLs', async function() {
     await bundle(__dirname + '/integration/html/index.html');
 

diff --git a/packages/core/parcel-bundler/test/integration/htmlnano-defaults-form/index.html b/packages/core/parcel-bundler/test/integration/htmlnano-defaults-form/index.html
@@ -0,0 +1,11 @@
+<!doctype html>
+<html>
+<head>
+  <title>Test</title>
+</head>
+<body>
+  <form>
+    <input type="text">
+  </form>
+</body>
+</html>