Avoid element update, from the model value change, if it comes from e…

…lement itself
GrapesJS · Oct 31, 2017 · 1279ad7 · 1279ad7
1 parent 2850554
commit 1279ad7
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/dist/grapes.min.js b/dist/grapes.min.js
diff --git a/src/editor/index.js b/src/editor/index.js
@@ -89,6 +89,7 @@
  *   style: '.txt-red{color: red}',
  * });
  */
+import $ from 'cash-dom';
 
 module.exports = config => {
   var c = config || {},
@@ -110,6 +111,8 @@ module.exports = config => {
 
   return {
 
+    $,
+
     /**
      * @property {EditorModel}
      * @private

diff --git a/src/style_manager/view/PropertyView.js b/src/style_manager/view/PropertyView.js
@@ -298,13 +298,17 @@ module.exports = Backbone.View.extend({
    * @param {Mixed} val  Value
    * @param {Object} opt  Options
    * */
-  modelValueChanged(e, val, opt) {
+  modelValueChanged(e, val, opt = {}) {
     const em = this.config.em;
     const model = this.model;
     const value = model.getFullValue();
     const target = this.getTarget();
     const onChange = this.onChange;
-    this.setRawValue(value);
+
+    // I don't need to update the input if the change comes from it
+    if (!opt.fromInput) {
+      this.setRawValue(value);
+    }
 
     // Check if component is allowed to be styled
     if (!target || !this.isTargetStylable() || !this.isComponentStylable()) {