Merge remote-tracking branch 'origin/staging/main' into staging/release

GitHub Action · GitHub Action · commit fa7264f332d6 · 2023-07-20T07:31:48.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## v1.14.9
+* Bump semver and editorconfig ([#2161](https://github.com/beautify-web/js-beautify/pull/2161))
+* Update editorconfig package ([#2160](https://github.com/beautify-web/js-beautify/issues/2160))
+* Allow to configure the "custom elements as inline elements" behavior ([#2113](https://github.com/beautify-web/js-beautify/issues/2113))
+
 ## v1.14.8
 * Require nodejs v12 or greater ([#2151](https://github.com/beautify-web/js-beautify/pull/2151))
 * CSS insideNonNestedAtRule generic variable  ([#2147](https://github.com/beautify-web/js-beautify/pull/2147))
diff --git a/README.md b/README.md
@@ -58,13 +58,13 @@ JS Beautifier is hosted on two CDN services: [cdnjs](https://cdnjs.com/libraries
 
 To pull the latest version from one of these services include one set of the script tags below in your document:
 ```html
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify.js"></script>
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify-css.js"></script>
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify-html.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify-css.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify-html.js"></script>
 
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify.min.js"></script>
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify-css.min.js"></script>
-<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify-html.min.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify.min.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify-css.min.js"></script>
+<script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify-html.min.js"></script>
 ```
 
 Example usage of a JS tag in html:
@@ -76,7 +76,7 @@ Example usage of a JS tag in html:
 
 . . .
 
-    <script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.8/beautify.min.js"></script>
+    <script src="https://cdnjs.cloudflare.com/ajax/libs/js-beautify/1.14.9/beautify.min.js"></script>
     <script src="script.js"></script>
   </body>
 </html>
@@ -371,6 +371,7 @@ HTML Beautifier Options:
   -M, --wrap-attributes-min-attrs    Minimum number of html tag attributes for force wrap attribute options [2]
   -i, --wrap-attributes-indent-size  Indent wrapped attributes to after N characters [indent-size] (ignored if wrap-attributes is "aligned")
   -d, --inline                       List of tags to be considered inline tags
+  --inline_custom_elements           Inline custom elements [true]
   -U, --unformatted                  List of tags (defaults to inline) that should not be reformatted
   -T, --content_unformatted          List of tags (defaults to pre) whose content should not be reformatted
   -E, --extra_liners                 List of tags (defaults to [head,body,/html] that should have an extra newline before them.
@@ -433,4 +434,4 @@ Thanks also to Jason Diamond, Patrick Hof, Nochum Sossonko, Andreas Schneider, D
 Vasilevsky, Vital Batmanov, Ron Baldwin, Gabriel Harrison, Chris J. Shull,
 Mathias Bynens, Vittorio Gambaletta and others.
 
-(README.md: js-beautify@1.14.8)
+(README.md: js-beautify@1.14.9)
diff --git a/js/src/cli.js b/js/src/cli.js
@@ -116,6 +116,7 @@ var path = require('path'),
         // HTML-only
         "max_char": Number, // obsolete since 1.3.5
         "inline": [String, Array],
+        "inline_custom_elements": [Boolean],
         "unformatted": [String, Array],
         "content_unformatted": [String, Array],
         "indent_inner_html": [Boolean],
@@ -168,6 +169,7 @@ var path = require('path'),
         "i": ["--wrap_attributes_indent_size"],
         "W": ["--max_char"], // obsolete since 1.3.5
         "d": ["--inline"],
+        // no shorthand for "inline_custom_elements"
         "U": ["--unformatted"],
         "T": ["--content_unformatted"],
         "I": ["--indent_inner_html"],
diff --git a/js/src/html/beautifier.js b/js/src/html/beautifier.js
@@ -656,7 +656,7 @@ Beautifier.prototype._get_tag_open_token = function(raw_token) { //function to g
 
   parser_token.is_unformatted = !parser_token.tag_complete && in_array(parser_token.tag_check, this._options.unformatted);
   parser_token.is_content_unformatted = !parser_token.is_empty_element && in_array(parser_token.tag_check, this._options.content_unformatted);
-  parser_token.is_inline_element = in_array(parser_token.tag_name, this._options.inline) || parser_token.tag_name.includes("-") || parser_token.tag_start_char === '{';
+  parser_token.is_inline_element = in_array(parser_token.tag_name, this._options.inline) || (this._options.inline_custom_elements && parser_token.tag_name.includes("-")) || parser_token.tag_start_char === '{';
 
   return parser_token;
 };
diff --git a/js/src/html/options.js b/js/src/html/options.js
@@ -61,6 +61,7 @@ function Options(options) {
     // obsolete inline tags
     'acronym', 'big', 'strike', 'tt'
   ]);
+  this.inline_custom_elements = this._get_boolean('inline_custom_elements', true);
   this.void_elements = this._get_array('void_elements', [
     // HTLM void elements - aka self-closing tags - aka singletons
     // https://www.w3.org/html/wg/drafts/html/master/syntax.html#void-elements
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "js-beautify",
-  "version": "1.14.8",
+  "version": "1.14.9",
   "description": "beautifier.io for node",
   "main": "js/index.js",
   "bin": {
@@ -50,7 +50,7 @@
   "browserslist": "ie 11",
   "dependencies": {
     "config-chain": "^1.1.13",
-    "editorconfig": "^0.15.3",
+    "editorconfig": "^1.0.3",
     "glob": "^8.1.0",
     "nopt": "^6.0.0"
   },
diff --git a/python/cssbeautifier/__version__.py b/python/cssbeautifier/__version__.py
@@ -1 +1 @@
-__version__ = "1.14.8"
+__version__ = "1.14.9"
diff --git a/python/cssbeautifier/tests/__init__.py b/python/cssbeautifier/tests/__init__.py
@@ -0,0 +1 @@
+# Empty file :)
diff --git a/python/jsbeautifier/__version__.py b/python/jsbeautifier/__version__.py
@@ -1 +1 @@
-__version__ = "1.14.8"
+__version__ = "1.14.9"
diff --git a/python/jsbeautifier/tests/generated/__init__.py b/python/jsbeautifier/tests/generated/__init__.py
@@ -0,0 +1 @@
+# Empty file :)
diff --git a/test/data/html/tests.js b/test/data/html/tests.js
@@ -3813,6 +3813,32 @@ exports.test_data = {
         '</span>'
       ]
     }]
+  }, {
+    name: "Disables custom elements inlining with inline_custom_elements=false",
+    description: "https://github.com/beautify-web/js-beautify/issues/2113",
+    options: [
+      { name: "inline_custom_elements", value: "false" }
+    ],
+    tests: [{
+      input: [
+        '<span>',
+        '    <span>',
+        '        <span>The time for this result is 1:02</span',
+        '        ><time-dot>.</time-dot',
+        '        ><time-decimals>27</time-decimals>',
+        '    </span>',
+        '</span>'
+      ],
+      output: [
+        '<span>',
+        '    <span>',
+        '        <span>The time for this result is 1:02</span>',
+        '        <time-dot>.</time-dot>',
+        '        <time-decimals>27</time-decimals>',
+        '    </span>',
+        '</span>'
+      ]
+    }]
   }, {
     name: "New Test Suite"
   }]
diff --git a/test/data/javascript/python.mustache b/test/data/javascript/python.mustache
@@ -367,7 +367,7 @@ class TestJSBeautifier(unittest.TestCase):
         test_fragment('foo {', 'foo {')
         test_fragment('return {', 'return {') # return needs the brace.
         test_fragment('return /* inline */ {', 'return /* inline */ {')
-        test_fragment('return;\n{', 'return; {')
+        test_fragment('return;\n{', 'return;\n{')
         bt("throw {}")
         bt("throw {\n    foo;\n}")
         bt('var foo = {}')
@@ -458,7 +458,7 @@ class TestJSBeautifier(unittest.TestCase):
         test_fragment('foo {', 'foo {')
         test_fragment('return {', 'return {') # return needs the brace.
         test_fragment('return /* inline */ {', 'return /* inline */ {')
-        test_fragment('return;\n{', 'return; {')
+        test_fragment('return;\n{', 'return;\n{')
         bt("throw {}")
         bt("throw {\n    foo;\n}")
         bt('var foo = {}')

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-__version__ = "1.14.8"`
	`1`	`+__version__ = "1.14.9"`