From 92f8a5be573784b20073a69c0fb2f87c7a2ae59b Mon Sep 17 00:00:00 2001 From: Brett Zamir Date: Sat, 4 Jan 2020 16:43:03 +0800 Subject: [PATCH] - Refactoring: Avoid redundant creation of text node for strings when passing to `append` --- cypress/integration/unit/draw.js | 6 +++--- cypress/integration/unit/recalculate.js | 2 +- editor/extensions/ext-imagelib.js | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/cypress/integration/unit/draw.js b/cypress/integration/unit/draw.js index 232c6373..450db7d1 100644 --- a/cypress/integration/unit/draw.js +++ b/cypress/integration/unit/draw.js @@ -69,19 +69,19 @@ describe('draw.Drawing', function () { const setupSVGWith3Layers = function (svgElem) { const layer1 = document.createElementNS(NS.SVG, 'g'); const layer1Title = document.createElementNS(NS.SVG, 'title'); - layer1Title.append(document.createTextNode(LAYER1)); + layer1Title.append(LAYER1); layer1.append(layer1Title); svgElem.append(layer1); const layer2 = document.createElementNS(NS.SVG, 'g'); const layer2Title = document.createElementNS(NS.SVG, 'title'); - layer2Title.append(document.createTextNode(LAYER2)); + layer2Title.append(LAYER2); layer2.append(layer2Title); svgElem.append(layer2); const layer3 = document.createElementNS(NS.SVG, 'g'); const layer3Title = document.createElementNS(NS.SVG, 'title'); - layer3Title.append(document.createTextNode(LAYER3)); + layer3Title.append(LAYER3); layer3.append(layer3Title); svgElem.append(layer3); diff --git a/cypress/integration/unit/recalculate.js b/cypress/integration/unit/recalculate.js index 1b98400b..aec3d447 100644 --- a/cypress/integration/unit/recalculate.js +++ b/cypress/integration/unit/recalculate.js @@ -89,7 +89,7 @@ describe('recalculate', function () { tspan.setAttribute('x', '200'); tspan.setAttribute('y', '150'); - const theText = document.createTextNode('Foo bar'); + const theText = 'Foo bar'; tspan.append(theText); elem.append(tspan); svg.append(elem); diff --git a/editor/extensions/ext-imagelib.js b/editor/extensions/ext-imagelib.js index ed251be9..eb9dd234 100644 --- a/editor/extensions/ext-imagelib.js +++ b/editor/extensions/ext-imagelib.js @@ -232,7 +232,7 @@ export default { $(this).html( $('').append( $('').attr('src', curMeta.preview_url), - document.createTextNode(title) + title ) ); } else { @@ -254,7 +254,7 @@ export default { if (curMeta && curMeta.preview_url) { entry = $('').append( $('').attr('src', curMeta.preview_url), - document.createTextNode(title) + title ); } else { entry = $('').attr('src', response);