From 6a61ca2879eb026f5bab8a1b555b2e1253a50e14 Mon Sep 17 00:00:00 2001 From: Arthur Almeida Date: Wed, 25 Apr 2018 08:44:30 -0300 Subject: [PATCH] refactor - rename variable --- src/code_manager/model/CssGenerator.js | 2 +- src/editor/config/config.js | 4 ++-- src/editor/model/Editor.js | 8 ++++---- test/specs/grapesjs/index.js | 8 ++++---- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/code_manager/model/CssGenerator.js b/src/code_manager/model/CssGenerator.js index 33d5c38a9..689f1932c 100644 --- a/src/code_manager/model/CssGenerator.js +++ b/src/code_manager/model/CssGenerator.js @@ -97,7 +97,7 @@ module.exports = require('backbone').Model.extend({ if ( this.compCls.indexOf(name) >= 0 || this.ids.indexOf(name) >= 0 || - opts.dumpUnusedStyles + opts.keepUnusedStyles ) { found = 1; } diff --git a/src/editor/config/config.js b/src/editor/config/config.js index a2320d5cb..5183d1575 100644 --- a/src/editor/config/config.js +++ b/src/editor/config/config.js @@ -281,6 +281,6 @@ module.exports = { // Texts textViewCode: 'Code', - // Dump unused styles within the editor - dumpUnusedStyles: 0 + // Keep unused styles within the editor + keepUnusedStyles: 0 }; diff --git a/src/editor/model/Editor.js b/src/editor/model/Editor.js index 983f079ba..a406000f5 100644 --- a/src/editor/model/Editor.js +++ b/src/editor/model/Editor.js @@ -302,9 +302,9 @@ module.exports = Backbone.Model.extend({ const config = this.config; const wrappesIsBody = config.wrappesIsBody; const avoidProt = opts.avoidProtected; - const dumpUnusedStyles = !isUndefined(opts.dumpUnusedStyles) - ? opts.dumpUnusedStyles - : config.dumpUnusedStyles; + const keepUnusedStyles = !isUndefined(opts.keepUnusedStyles) + ? opts.keepUnusedStyles + : config.keepUnusedStyles; const cssc = this.get('CssComposer'); const wrp = this.get('DomComponents').getComponent(); const protCss = !avoidProt ? config.protectedCss : ''; @@ -314,7 +314,7 @@ module.exports = Backbone.Model.extend({ this.get('CodeManager').getCode(wrp, 'css', { cssc, wrappesIsBody, - dumpUnusedStyles + keepUnusedStyles }) ); }, diff --git a/test/specs/grapesjs/index.js b/test/specs/grapesjs/index.js index f89d0d666..099262459 100644 --- a/test/specs/grapesjs/index.js +++ b/test/specs/grapesjs/index.js @@ -377,22 +377,22 @@ describe('GrapesJS', () => { expect(editor.Commands.get('export-template').test).toEqual(1); }); - it('Dump unused css classes/selectors', () => { + it('Keep unused css classes/selectors option for getCSS method', () => { config.fromElement = 1; config.storageManager = { type: 0 }; fixture.innerHTML = documentEl; const editor = obj.init(config); - const css = editor.getCss({ dumpUnusedStyles: 1 }); + const css = editor.getCss({ keepUnusedStyles: 1 }); const protCss = editor.getConfig().protectedCss; expect(editor.getStyle().length).toEqual(2); expect(css).toEqual(`${protCss}.test2{color:red;}.test3{color:blue;}`); }); - it('Dump unused css classes/selectors using the init option', () => { + it('Keep unused css classes/selectors option for init method', () => { config.fromElement = 1; config.storageManager = { type: 0 }; fixture.innerHTML = documentEl; - const editor = obj.init({ ...config, dumpUnusedStyles: 1 }); + const editor = obj.init({ ...config, keepUnusedStyles: 1 }); const css = editor.getCss(); const protCss = editor.getConfig().protectedCss; expect(editor.getStyle().length).toEqual(2);