Pull request #1477: KERNEL-5172 editor代码整理

Merge in VISUAL/fineui from ~TELLER/fineui:master to master

* commit '11ef5ac737e33f71670424a711eeb8aa068b52dc':
  chore: 修改dev命令
  refactor: 修改demo
  KERNEL-5172 fix: editor修改

demo/js/base/editor/demo.editor.js

@@ -6,7 +6,7 @@ Demo.Editor = BI.inherit(BI.Widget, {
         var editor1 = BI.createWidget({
             type: "bi.editor",
             cls: "bi-border",
-            watermark: "alert信息显示在下面",
+            watermark: "报错信息显示在控件上方",
             errorText: "字段不可重名!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!",
             width: 200,
             height: 24
@@ -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",

package.json

@@ -53,7 +53,7 @@
     "webpack-merge": "4.2.1"
   },
   "scripts": {
-    "webpack:dev": "webpack-dev-server -p --progress --host 0.0.0.0 --config=webpack/webpack.dev.js --mode development",
+    "webpack:dev": "node --max_old_space_size=4096 node_modules/.bin/webpack-dev-server -p --progress --host 0.0.0.0 --config=webpack/webpack.dev.js --mode development",
     "webpack:prod": "webpack -p --progress --config=webpack/webpack.prod.js --mode production",
     "start": "node server.js",
     "build": "npm run webpack:prod && tsc",

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,7 +236,12 @@ BI.Editor = BI.inherit(BI.Single, {
     },
 
     setWaterMark: function(v) {
+        if (!BI.isKey(v)) {
+            return;
+        }
+
         this.options.watermark = v;
+
         if (BI.isNull(this.watermark)) {
             this._assertWaterMark();
             BI.createWidget({
@@ -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) {