html-minifier 3.5.4 → 3.5.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -23,17 +23,17 @@ How does HTMLMinifier compare to other solutions — [HTML Minifier from Will Pe
23
23
  | Site | Original size *(KB)* | HTMLMinifier | minimize | Will Peavy | htmlcompressor.com |
24
24
  | ---------------------------------------------------------------------------- |:--------------------:| ------------:| --------:| ----------:| ------------------:|
25
25
  | [Google](https://www.google.com/) | 45 | **42** | 45 | 47 | 45 |
26
- | [HTMLMinifier](https://github.com/kangax/html-minifier) | 131 | **102** | 110 | 114 | 109 |
27
- | [Amazon](https://www.amazon.co.uk/) | 204 | **171** | 195 | 199 | n/a |
28
- | [New York Times](https://www.nytimes.com/) | 233 | **159** | 182 | 178 | 166 |
29
- | [Stack Overflow](https://stackoverflow.com/) | 246 | **193** | 202 | 210 | 199 |
26
+ | [HTMLMinifier](https://github.com/kangax/html-minifier) | 133 | **104** | 112 | 116 | 111 |
27
+ | [Amazon](https://www.amazon.co.uk/) | 203 | **171** | 195 | 199 | n/a |
28
+ | [New York Times](https://www.nytimes.com/) | 233 | **158** | 182 | 178 | 166 |
29
+ | [Stack Overflow](https://stackoverflow.com/) | 239 | **186** | 195 | 203 | 192 |
30
30
  | [Bootstrap CSS](https://getbootstrap.com/docs/3.3/css/) | 272 | **260** | 269 | 229 | 269 |
31
- | [Twitter](https://twitter.com/) | 291 | **216** | 257 | 280 | 257 |
32
- | [BBC](https://www.bbc.co.uk/) | 311 | **256** | 302 | 310 | 294 |
31
+ | [BBC](https://www.bbc.co.uk/) | 279 | **229** | 271 | 278 | 262 |
32
+ | [Twitter](https://twitter.com/) | 298 | **223** | 263 | 287 | 263 |
33
33
  | [Wikipedia](https://en.wikipedia.org/wiki/President_of_the_United_States) | 469 | **437** | 453 | 468 | 453 |
34
- | [NBC](https://www.nbc.com/) | 663 | **626** | 659 | 663 | n/a |
34
+ | [NBC](https://www.nbc.com/) | 665 | **627** | 661 | 664 | n/a |
35
35
  | [Eloquent Javascript](https://eloquentjavascript.net/1st_edition/print.html) | 870 | **815** | 840 | 864 | n/a |
36
- | [ES6 table](https://kangax.github.io/compat-table/es6/) | 4227 | **3585** | 3998 | n/a | n/a |
36
+ | [ES6 table](https://kangax.github.io/compat-table/es6/) | 4228 | **3586** | 3999 | n/a | n/a |
37
37
  | [ES6 draft](https://tc39.github.io/ecma262/) | 5701 | **5087** | 5236 | n/a | n/a |
38
38
 
39
39
  ## Options Quick Reference
package/package.json CHANGED
@@ -1,7 +1,7 @@
1
1
  {
2
2
  "name": "html-minifier",
3
3
  "description": "Highly configurable, well-tested, JavaScript-based HTML minifier.",
4
- "version": "3.5.4",
4
+ "version": "3.5.5",
5
5
  "keywords": [
6
6
  "cli",
7
7
  "compress",
@@ -66,7 +66,7 @@
66
66
  "devDependencies": {
67
67
  "grunt": "1.0.x",
68
68
  "grunt-browserify": "5.2.x",
69
- "grunt-contrib-uglify": "3.0.x",
69
+ "grunt-contrib-uglify": "3.1.x",
70
70
  "gruntify-eslint": "4.0.x",
71
71
  "phantomjs-prebuilt": "2.1.x",
72
72
  "qunitjs": "2.x"
@@ -8,20 +8,16 @@ var TokenChain = require('./tokenchain');
8
8
  var UglifyJS = require('uglify-js');
9
9
  var utils = require('./utils');
10
10
 
11
- var trimWhitespace = String.prototype.trim ? function(str) {
11
+ function trimWhitespace(str) {
12
12
  if (typeof str !== 'string') {
13
13
  return str;
14
14
  }
15
- return str.trim();
16
- } : function(str) {
17
- if (typeof str !== 'string') {
18
- return str;
19
- }
20
- return str.replace(/^\s+/, '').replace(/\s+$/, '');
21
- };
15
+ return str.replace(/^[ \n\r\t\f]+/, '').replace(/[ \n\r\t\f]+$/, '');
16
+ }
22
17
 
23
18
  function collapseWhitespaceAll(str) {
24
- return str && str.replace(/\s+/g, function(spaces) {
19
+ // Non-breaking space is specifically handled inside the replacer function here:
20
+ return str && str.replace(/[ \n\r\t\f\xA0]+/g, function(spaces) {
25
21
  return spaces === '\t' ? '\t' : spaces.replace(/(^|\xA0+)[^\xA0]+/g, '$1 ');
26
22
  });
27
23
  }
@@ -30,17 +26,18 @@ function collapseWhitespace(str, options, trimLeft, trimRight, collapseAll) {
30
26
  var lineBreakBefore = '', lineBreakAfter = '';
31
27
 
32
28
  if (options.preserveLineBreaks) {
33
- str = str.replace(/^\s*?[\n\r]\s*/, function() {
29
+ str = str.replace(/^[ \n\r\t\f]*?[\n\r][ \n\r\t\f]*/, function() {
34
30
  lineBreakBefore = '\n';
35
31
  return '';
36
- }).replace(/\s*?[\n\r]\s*$/, function() {
32
+ }).replace(/[ \n\r\t\f]*?[\n\r][ \n\r\t\f]*$/, function() {
37
33
  lineBreakAfter = '\n';
38
34
  return '';
39
35
  });
40
36
  }
41
37
 
42
38
  if (trimLeft) {
43
- str = str.replace(/^\s+/, function(spaces) {
39
+ // Non-breaking space is specifically handled inside the replacer function here:
40
+ str = str.replace(/^[ \n\r\t\f\xA0]+/, function(spaces) {
44
41
  var conservative = !lineBreakBefore && options.conservativeCollapse;
45
42
  if (conservative && spaces === '\t') {
46
43
  return '\t';
@@ -50,7 +47,8 @@ function collapseWhitespace(str, options, trimLeft, trimRight, collapseAll) {
50
47
  }
51
48
 
52
49
  if (trimRight) {
53
- str = str.replace(/\s+$/, function(spaces) {
50
+ // Non-breaking space is specifically handled inside the replacer function here:
51
+ str = str.replace(/[ \n\r\t\f\xA0]+$/, function(spaces) {
54
52
  var conservative = !lineBreakAfter && options.conservativeCollapse;
55
53
  if (conservative && spaces === '\t') {
56
54
  return '\t';
@@ -1253,7 +1251,7 @@ function minify(value, options, partialMarkup) {
1253
1251
  return collapseWhitespace(chunk, {
1254
1252
  preserveLineBreaks: options.preserveLineBreaks,
1255
1253
  conservativeCollapse: !options.trimCustomFragments
1256
- }, /^\s/.test(chunk), /\s$/.test(chunk));
1254
+ }, /^[ \n\r\t\f]/.test(chunk), /[ \n\r\t\f]$/.test(chunk));
1257
1255
  }
1258
1256
  return chunk;
1259
1257
  });