From c067965ec122810cb4997073d5939a60366375e4 Mon Sep 17 00:00:00 2001
From: iapyang <yklorse@gmail.com>
Date: Fri, 28 Aug 2020 14:41:23 +0800
Subject: [PATCH] =?UTF-8?q?KERNEL-5172=20fix:=20editor=E4=BF=AE=E6=94=B9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 demo/js/base/editor/demo.editor.js |  4 +--
 src/base/single/editor/editor.js   | 52 +++++++++++++-----------------
 2 files changed, 25 insertions(+), 31 deletions(-)

diff --git a/demo/js/base/editor/demo.editor.js b/demo/js/base/editor/demo.editor.js
index 6268cc54d..208b8f596 100644
--- a/demo/js/base/editor/demo.editor.js
+++ b/demo/js/base/editor/demo.editor.js
@@ -16,7 +16,7 @@ Demo.Editor = BI.inherit(BI.Widget, {
         });
         var editor2 = BI.createWidget({
             type: "bi.editor",
-            cls: "mvc-border",
+            cls: "bi-border",
             watermark: "输入'a'会有错误信息",
             disabled: true,
             errorText: "字段不可重名",
@@ -32,7 +32,7 @@ Demo.Editor = BI.inherit(BI.Widget, {
         });
         var editor3 = BI.createWidget({
             type: "bi.editor",
-            cls: "mvc-border",
+            cls: "bi-border",
             watermark: "输入'a'会有错误信息且回车键不能退出编辑",
             errorText: "字段不可重名",
             value: "a",
diff --git a/src/base/single/editor/editor.js b/src/base/single/editor/editor.js
index 793c7da6e..a5c455ed1 100644
--- a/src/base/single/editor/editor.js
+++ b/src/base/single/editor/editor.js
@@ -6,6 +6,7 @@
 BI.Editor = BI.inherit(BI.Single, {
     _defaultConfig: function () {
         var conf = BI.Editor.superclass._defaultConfig.apply(this, arguments);
+
         return BI.extend(conf, {
             baseCls: "bi-editor bi-focus-shadow",
             hgap: 4,
@@ -21,7 +22,7 @@ BI.Editor = BI.inherit(BI.Single, {
             quitChecker: BI.emptyFn,
             allowBlank: false,
             watermark: "",
-            errorText: ""
+            errorText: "",
         });
     },
 
@@ -46,27 +47,6 @@ BI.Editor = BI.inherit(BI.Single, {
             padding: "0",
             margin: "0"
         });
-        if (BI.isKey(this.options.watermark)) {
-            this._assertWaterMark();
-        }
-
-        var _items = [];
-        if (this.watermark) {
-            _items.push({
-                el: this.watermark,
-                left: 3,
-                right: 3,
-                top: 0,
-                bottom: 0
-            });
-        }
-        _items.push({
-            el: this.editor,
-            left: 0,
-            right: 0,
-            top: 0,
-            bottom: 0
-        });
 
         var items = [{
             el: {
@@ -74,7 +54,13 @@ BI.Editor = BI.inherit(BI.Single, {
                 ref: function(_ref) {
                     self.contentWrapper = _ref;
                 },
-                items: _items
+                items: [{
+                    el: this.editor,
+                    left: 0,
+                    right: 0,
+                    top: 0,
+                    bottom: 0
+                }]
             },
             left: o.hgap + o.lgap,
             right: o.hgap + o.rgap,
@@ -87,6 +73,9 @@ BI.Editor = BI.inherit(BI.Single, {
             element: this,
             items: items
         });
+
+        this.setWaterMark(this.options.watermark);
+
         this.editor.on(BI.Controller.EVENT_CHANGE, function () {
             self.fireEvent(BI.Controller.EVENT_CHANGE, arguments);
         });
@@ -199,7 +188,7 @@ BI.Editor = BI.inherit(BI.Single, {
                 type: "bi.label",
                 cls: "bi-water-mark",
                 text: this.options.watermark,
-                height: o.height - 2 * (o.vgap + o.tgap),
+                height: o.height - 2 * o.vgap - o.tgap,
                 whiteSpace: "nowrap",
                 textAlign: "left"
             });
@@ -247,8 +236,13 @@ BI.Editor = BI.inherit(BI.Single, {
     },
 
     setWaterMark: function(v) {
+        if (!BI.isKey(v)) {
+            return;
+        }
+
         this.options.watermark = v;
-        if(BI.isNull(this.watermark)) {
+
+        if (BI.isNull(this.watermark)) {
             this._assertWaterMark();
             BI.createWidget({
                 type: "bi.absolute",
@@ -258,11 +252,11 @@ BI.Editor = BI.inherit(BI.Single, {
                     left: 3,
                     right: 3,
                     top: 0,
-                    bottom: 0
-                }]
-            })
+                    bottom: 0,
+                }],
+            });
         }
-        BI.isKey(v) && this.watermark.setText(v);
+        this.watermark.setText(v);
     },
 
     _setErrorVisible: function (b) {