From 32b8e89b13573198f9dde822844991b0362e1a4d Mon Sep 17 00:00:00 2001
From: Paulo Gustavo Veiga <pveiga@wisemapping.com>
Date: Sat, 10 Dec 2022 12:31:30 -0800
Subject: [PATCH] Migrate topic style.

---
 .../src/components/IconGroupRemoveTip.ts      |   1 +
 .../mindplot/src/components/TopicFeature.js   |   4 +-
 .../{TopicStyle.js => TopicStyle.ts}          | 115 ++++++++++--------
 3 files changed, 68 insertions(+), 52 deletions(-)
 rename packages/mindplot/src/components/{TopicStyle.js => TopicStyle.ts} (76%)

diff --git a/packages/mindplot/src/components/IconGroupRemoveTip.ts b/packages/mindplot/src/components/IconGroupRemoveTip.ts
index e186cc79..82886d54 100644
--- a/packages/mindplot/src/components/IconGroupRemoveTip.ts
+++ b/packages/mindplot/src/components/IconGroupRemoveTip.ts
@@ -67,6 +67,7 @@ class IconGroupRemoveTip {
 
     if (this._activeIcon) {
       const widget = this._widget;
+
       const close = () => {
         this._activeIcon = null;
         this._group.removeChild(widget);
diff --git a/packages/mindplot/src/components/TopicFeature.js b/packages/mindplot/src/components/TopicFeature.js
index 5e79625f..ce319b80 100644
--- a/packages/mindplot/src/components/TopicFeature.js
+++ b/packages/mindplot/src/components/TopicFeature.js
@@ -21,6 +21,8 @@ import EmojiCharIcon from './EmojiCharIcon';
 import SvgImageIcon from './SvgImageIcon';
 import LinkIcon from './LinkIcon';
 import NoteIcon from './NoteIcon';
+import Topic from './Topic';
+import FeatureModel from './model/FeatureModel';
 
 const TopicFeatureFactory = {
   /** the icon object */
@@ -46,7 +48,7 @@ const TopicFeatureFactory = {
     icon: NoteIcon,
   },
 
-  createIcon(topic, model, readOnly) {
+  createIcon(topic: Topic, model: FeatureModel, readOnly: boolean) {
     $assert(topic, 'topic can not be null');
     $assert(model, 'model can not be null');
 
diff --git a/packages/mindplot/src/components/TopicStyle.js b/packages/mindplot/src/components/TopicStyle.ts
similarity index 76%
rename from packages/mindplot/src/components/TopicStyle.js
rename to packages/mindplot/src/components/TopicStyle.ts
index 37cb4beb..5f4e07bd 100644
--- a/packages/mindplot/src/components/TopicStyle.js
+++ b/packages/mindplot/src/components/TopicStyle.ts
@@ -15,59 +15,26 @@
  *   See the License for the specific language governing permissions and
  *   limitations under the License.
  */
-import { $assert, $defined } from '@wisemapping/core-js';
+import { $assert } from '@wisemapping/core-js';
 import { $msg } from './Messages';
 import { TopicShape } from './model/INodeModel';
+import Topic from './Topic';
 
-class TopicStyle {
-  static _getStyles(topic) {
-    $assert(topic, 'topic can not be null');
+type TopicStyleType = {
+  borderColor: string;
+  backgroundColor: string;
+  fontStyle: {
+    font: string;
+    size: number;
+    style: string;
+    weight: string;
+    color: string;
+  };
+  msgKey: string;
+  shapeType: string;
+};
 
-    let result;
-    if (topic.isCentralTopic()) {
-      result = TopicStyle.STYLES.CENTRAL_TOPIC;
-    } else {
-      const targetTopic = topic.getOutgoingConnectedTopic();
-      if ($defined(targetTopic)) {
-        if (targetTopic.isCentralTopic()) {
-          result = TopicStyle.STYLES.MAIN_TOPIC;
-        } else {
-          result = TopicStyle.STYLES.SUB_TOPIC;
-        }
-      } else {
-        result = TopicStyle.STYLES.ISOLATED_TOPIC;
-      }
-    }
-    return result;
-  }
-
-  static defaultText(topic) {
-    const { msgKey } = this._getStyles(topic);
-    return $msg(msgKey);
-  }
-
-  static defaultFontStyle(topic) {
-    return this._getStyles(topic).fontStyle;
-  }
-
-  static defaultBackgroundColor(topic) {
-    return this._getStyles(topic).backgroundColor;
-  }
-
-  static defaultBorderColor(topic) {
-    return this._getStyles(topic).borderColor;
-  }
-
-  static getInnerPadding(topic) {
-    return Math.round(topic.getTextShape().getFontHeight() * 0.5);
-  }
-
-  static defaultShapeType(topic) {
-    return this._getStyles(topic).shapeType;
-  }
-}
-
-TopicStyle.STYLES = {
+const TopicDefaultStyles = {
   CENTRAL_TOPIC: {
     borderColor: 'rgb(57,113,177)',
     backgroundColor: 'rgb(80,157,192)',
@@ -81,7 +48,6 @@ TopicStyle.STYLES = {
     msgKey: 'CENTRAL_TOPIC',
     shapeType: TopicShape.ROUNDED_RECT,
   },
-
   MAIN_TOPIC: {
     borderColor: 'rgb(2,59,185)',
     backgroundColor: 'rgb(224,229,239)',
@@ -95,7 +61,6 @@ TopicStyle.STYLES = {
     msgKey: 'MAIN_TOPIC',
     shapeType: TopicShape.LINE,
   },
-
   SUB_TOPIC: {
     borderColor: 'rgb(2,59,185)',
     backgroundColor: 'rgb(224,229,239)',
@@ -125,4 +90,52 @@ TopicStyle.STYLES = {
   },
 };
 
+class TopicStyle {
+  static _getStyles(topic: Topic): TopicStyleType {
+    $assert(topic, 'topic can not be null');
+
+    let result: TopicStyleType;
+    if (topic.isCentralTopic()) {
+      result = TopicDefaultStyles.CENTRAL_TOPIC;
+    } else {
+      const targetTopic = topic.getOutgoingConnectedTopic();
+      if (targetTopic) {
+        if (targetTopic.isCentralTopic()) {
+          result = TopicDefaultStyles.MAIN_TOPIC;
+        } else {
+          result = TopicDefaultStyles.SUB_TOPIC;
+        }
+      } else {
+        result = TopicDefaultStyles.ISOLATED_TOPIC;
+      }
+    }
+    return result;
+  }
+
+  static defaultText(topic: Topic) {
+    const { msgKey } = this._getStyles(topic);
+    return $msg(msgKey);
+  }
+
+  static defaultFontStyle(topic: Topic) {
+    return this._getStyles(topic).fontStyle;
+  }
+
+  static defaultBackgroundColor(topic: Topic) {
+    return this._getStyles(topic).backgroundColor;
+  }
+
+  static defaultBorderColor(topic: Topic) {
+    return this._getStyles(topic).borderColor;
+  }
+
+  static getInnerPadding(topic: Topic) {
+    return Math.round(topic.getTextShape().getFontHeight() * 0.5);
+  }
+
+  static defaultShapeType(topic) {
+    return this._getStyles(topic).shapeType;
+  }
+}
+
 export default TopicStyle;