From db246ccefe6864f6ad5641307894658ddbc544ff Mon Sep 17 00:00:00 2001
From: Joeri Exelmans <joeri.exelmans@gmail.com>
Date: Sun, 5 Oct 2025 15:47:42 +0200
Subject: [PATCH] better error messages

---
 src/VisualEditor/VisualEditor.tsx | 2 +-
 src/VisualEditor/parser.ts        | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/VisualEditor/VisualEditor.tsx b/src/VisualEditor/VisualEditor.tsx
index af7e7a9..cd5e546 100644
--- a/src/VisualEditor/VisualEditor.tsx
+++ b/src/VisualEditor/VisualEditor.tsx
@@ -645,7 +645,7 @@ export function ArrowSVG(props: {arrow: Arrow, selected: string[], errors: strin
       data-parts="start end"
     />
 
-    {props.errors.length>0 && <text className="error" x={(start.x+end.x)/2} y={(start.y+end.y)/2} data-uid={uid} data-parts="start end">{props.errors.join(' ')}</text>}
+    {props.errors.length>0 && <text className="error" x={(start.x+end.x)/2+5} y={(start.y+end.y)/2} data-uid={uid} data-parts="start end">{props.errors.join(' ')}</text>}
 
     <line
       className="lineHelper"
diff --git a/src/VisualEditor/parser.ts b/src/VisualEditor/parser.ts
index cf8fbd3..7fd221a 100644
--- a/src/VisualEditor/parser.ts
+++ b/src/VisualEditor/parser.ts
@@ -59,7 +59,7 @@ export function parseStatechart(state: VisualEditorState): [Statechart, [string,
     if (!srcUID) {
       if (!tgtUID) {
         // dangling edge - todo: display error...
-        errorShapes.push([arr.uid, "Dangling edge"]);
+        errorShapes.push([arr.uid, "dangling"]);
       }
       else {
         // target but no source, so we treat is as an 'initial' marking
@@ -76,7 +76,7 @@ export function parseStatechart(state: VisualEditorState): [Statechart, [string,
     }
     else {
       if (!tgtUID) {
-        errorShapes.push([arr.uid, "Needs target"]);
+        errorShapes.push([arr.uid, "no target"]);
       }
       else {
         // add transition
@@ -100,10 +100,10 @@ export function parseStatechart(state: VisualEditorState): [Statechart, [string,
   for (const state of uid2State.values()) {
     if (state.kind === "or") {
       if (state.initial.length > 1) {
-        errorShapes.push(...state.initial.map(([uid,childState])=>[uid,"OR-state can only have 1 initial state"] as [string, string]));
+        errorShapes.push(...state.initial.map(([uid,childState])=>[uid,"multiple initial states"] as [string, string]));
       }
       else if (state.initial.length === 0) {
-        errorShapes.push([state.uid, "Needs initial state"]);
+        errorShapes.push([state.uid, "no initial state"]);
       }
     }
   }