html-minifier - npm Package Compare versions

Comparing version 3.2.2 to 3.2.3

package.json

 {
   "name": "html-minifier",
   "description": "Highly configurable, well-tested, JavaScript-based HTML minifier.",
-  "version": "3.2.2",
+  "version": "3.2.3",
   "keywords": [
@@ -6,0 +6,0 @@ "cli",

README.md

@@ -25,18 +25,20 @@ # HTMLMinifier
 | --------------------------------------------------------------------------- |:--------------------:| ------------:| --------:| ----------:| ------------------:|
-| [Google](https://www.google.com/)                                           | 44                   | **43**       | 44       | 46         | 45                 |
-| [HTMLMinifier](https://github.com/kangax/html-minifier)                     | 122                  | **96**       | 104      | 108        | 103                |
-| [CNN](http://www.cnn.com/)                                                  | 129                  | **118**      | 126      | 127        | 122                |
-| [Amazon](http://www.amazon.co.uk/)                                          | 170                  | **138**      | 162      | 165        | n/a                |
-| [BBC](http://www.bbc.co.uk/)                                                | 185                  | **151**      | 178      | 183        | 174                |
-| [New York Times](http://www.nytimes.com/)                                   | 187                  | **124**      | 137      | 139        | 132                |
-| [Stack Overflow](http://stackoverflow.com/)                                 | 245                  | **189**      | 198      | 207        | 196                |
+| [Google](https://www.google.com/)                                           | 44                   | **42**       | 44       | 46         | 45                 |
+| [HTMLMinifier](https://github.com/kangax/html-minifier)                     | 122                  | **96**       | 103      | 107        | 103                |
+| [CNN](http://www.cnn.com/)                                                  | 132                  | **121**      | 128      | 129        | 124                |
+| [BBC](http://www.bbc.co.uk/)                                                | 178                  | **146**      | 172      | 176        | 168                |
+| [Amazon](http://www.amazon.co.uk/)                                          | 185                  | **152**      | 177      | 179        | n/a                |
+| [New York Times](http://www.nytimes.com/)                                   | 195                  | **128**      | 143      | 145        | 135                |
+| [Stack Overflow](http://stackoverflow.com/)                                 | 244                  | **188**      | 198      | 206        | 196                |
 | [Bootstrap CSS](http://getbootstrap.com/css/)                               | 276                  | **264**      | 273      | 231        | 274                |
-| [Wikipedia](https://en.wikipedia.org/wiki/President_of_the_United_States)   | 501                  | **456**      | 484      | 499        | n/a                |
-| [NBC](http://www.nbc.com/)                                                  | 550                  | **528**      | 548      | 550        | n/a                |
+| [Wikipedia](https://en.wikipedia.org/wiki/President_of_the_United_States)   | 499                  | **454**      | 482      | 498        | n/a                |
+| [NBC](http://www.nbc.com/)                                                  | 548                  | **526**      | 546      | 548        | n/a                |
 | [Eloquent Javascript](http://eloquentjavascript.net/1st_edition/print.html) | 870                  | **815**      | 840      | 864        | n/a                |
-| [ES6 table](http://kangax.github.io/compat-table/es6/)                      | 4010                 | **3374**     | 3781     | n/a        | n/a                |
-| [ES6 draft](https://tc39.github.io/ecma262/)                                | 5156                 | **4594**     | 4726     | n/a        | n/a                |
+| [ES6 table](http://kangax.github.io/compat-table/es6/)                      | 4113                 | **3459**     | 3878     | n/a        | n/a                |
+| [ES6 draft](https://tc39.github.io/ecma262/)                                | 5157                 | **4595**     | 4727     | n/a        | n/a                |
 
 ## Options Quick Reference
 
+Most of the options are disabled by default.
+
 | Option                         | Description     | Default |
@@ -70,3 +72,3 @@ |--------------------------------|-----------------|---------|
 | `removeComments`               | [Strip HTML comments](http://perfectionkills.com/experimenting-with-html-minifier/#remove_comments) | `false` |
-| `removeEmptyAttributes`        | [Remove all attributes with whitespace-only values](http://perfectionkills.com/experimenting-with-html-minifier/#remove_empty_or_blank_attributes) | `false` (could be true, Function(attrName, tag)) |
+| `removeEmptyAttributes`        | [Remove all attributes with whitespace-only values](http://perfectionkills.com/experimenting-with-html-minifier/#remove_empty_or_blank_attributes) | `false` (could be `true`, `Function(attrName, tag)`) |
 | `removeEmptyElements`          | [Remove all elements with empty contents](http://perfectionkills.com/experimenting-with-html-minifier/#remove_empty_elements) | `false` |
@@ -73,0 +75,0 @@ | `removeOptionalTags`           | [Remove optional tags](http://perfectionkills.com/experimenting-with-html-minifier/#remove_optional_tags) | `false` |

src/htmlminifier.js

@@ -551,3 +551,3 @@ 'use strict';
 
-function buildAttr(normalized, hasUnarySlash, options, isLast) {
+function buildAttr(normalized, hasUnarySlash, options, isLast, uidAttr) {
   var attrName = normalized.name,
@@ -560,4 +560,4 @@ attrValue = normalized.value,
 
-  if (typeof attrValue !== 'undefined' && !options.removeAttributeQuotes ||
-      !canRemoveAttributeQuotes(attrValue)) {
+  if (typeof attrValue !== 'undefined' && (!options.removeAttributeQuotes ||
+      ~attrValue.indexOf(uidAttr) || !canRemoveAttributeQuotes(attrValue))) {
     if (!options.preventAttributesEscaping) {
@@ -1035,3 +1035,3 @@ if (typeof options.quoteCharacter === 'undefined') {
         if (normalized) {
-          parts.unshift(buildAttr(normalized, hasUnarySlash, options, isLast));
+          parts.unshift(buildAttr(normalized, hasUnarySlash, options, isLast, uidAttr));
           isLast = false;
@@ -1038,0 +1038,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

88144

increased by0.11%

Number of lines in readme file

160

increased by1.27%

No dependency changes