do not encode entities in attribute values (#150)

Author: ehrencrona

src/lib/elements.ts

@@ -76,7 +76,7 @@ export const inlineElements: TagName[] = [
 /**
  * HTML attributes that we may safely reformat (trim whitespace, add or remove newlines)
  */
-export const formattableAttributes = [
+export const formattableAttributes: string[] = [
   // None at the moment
   // Prettier HTML does not format attributes at all
   // and to be consistent we leave this array empty for now

src/print/index.ts

@@ -18,6 +18,7 @@ import {
     isOrCanBeConvertedToShorthand,
     getNextNode,
     getNodeEnd,
+    getUnencodedText
 } from './node-helpers';
 import {
     isLine,
@@ -146,7 +147,7 @@ export function print(path: FastPath, options: ParserOptions, print: PrintFn): D
                          * allow for flexible grouping of HTML tags in a particular indentation level,
                          * and is similar to how vanilla HTML is handled in Prettier core.
                          */
-                        keepIfLonely: /\n\r?\s*\n\r?/.test(node.raw || node.data),
+                        keepIfLonely: /\n\r?\s*\n\r?/.test(getUnencodedText(node)),
                     };
                 }
 
@@ -156,9 +157,9 @@ export function print(path: FastPath, options: ParserOptions, print: PrintFn): D
                  * until this node's current line is out of room, at which `fill` will break at the
                  * most convenient instance of `line`.
                  */
-                return fill(splitTextToDocs(node.raw || node.data));
+                return fill(splitTextToDocs(getUnencodedText(node)));
             } else {
-                return node.data;
+                return getUnencodedText(node);
             }
         case 'Element':
         case 'InlineComponent':

src/print/node-helpers.ts

@@ -53,7 +53,7 @@ export function canBreakBefore(node: Node) {
 export function isInlineNode(node: Node): boolean {
     switch (node.type) {
         case 'Text':
-            const text = node.raw || node.data;
+            const text = getUnencodedText(node);
             const isAllWhitespace = text.trim() === '';
 
             return !isAllWhitespace || text === '';
@@ -105,7 +105,7 @@ export function getNextNode(path: FastPath): Node | undefined {
 }
 
 /**
- * Returns the position that used to be the end of the node before the embed elements were cut out. 
+ * Returns the position that used to be the end of the node before the embed elements were cut out.
  */
 export function getNodeEnd(node: Node, path: FastPath) {
     const root = path.stack[0];
@@ -122,7 +122,7 @@ export function getNodeEnd(node: Node, path: FastPath) {
 }
 
 export function isEmptyNode(node: Node): boolean {
-    return node.type === 'Text' && (node.raw || node.data).trim() === '';
+    return node.type === 'Text' && getUnencodedText(node).trim() === '';
 }
 
 export function isIgnoreDirective(node: Node | undefined | null): boolean {
@@ -211,3 +211,8 @@ export function isOrCanBeConvertedToShorthand(node: AttributeNode): boolean {
 
     return false;
 }
+
+export function getUnencodedText(node: TextNode) {
+    // `raw` will contain HTML entities in unencoded form
+    return node.raw || node.data;
+}

test/printer/samples/text-html-entities.html

@@ -1 +1 @@
-&
+<i data-icon="&#xE04E;">&amp;</i>