From 031c0b7b21c91f15b3d5ee64467b3e6e13aa9023 Mon Sep 17 00:00:00 2001
From: Knut Sveidqvist <knsv@sveido.com>
Date: Thu, 10 Oct 2019 20:55:27 +0200
Subject: [PATCH] #945 Increasing size, handling forks and joins in multiple
 directions

---
 dist/index.html                     | 24 +++++++++++++++++++++++-
 src/diagrams/state/shapes.js        |  4 +++-
 src/diagrams/state/stateDb.js       |  1 +
 src/diagrams/state/stateRenderer.js | 21 +++++++++++++--------
 4 files changed, 40 insertions(+), 10 deletions(-)

diff --git a/dist/index.html b/dist/index.html
index f1ce7fa8e..73f6ff154 100644
--- a/dist/index.html
+++ b/dist/index.html
@@ -420,8 +420,30 @@ class Class10 {
   </div>
   <div class="mermaid">
     stateDiagram
-      s1
+      State1
       </div>
+
+      <hr>
+      <div class="mermaid">
+        stateDiagram
+        [*] --> First
+        state First {
+            [*] --> second
+            second --> [*]
+        }
+          </div>
+  <div class="mermaid">
+    stateDiagram
+        State1: The state with a note
+        note right of State1
+            Important information! You can write
+            notes.
+        end note
+        State1 --> State2
+        note left of State2 : This is the note to the left.
+      </div>
+
+
   <script src="./mermaid.js"></script>
   <script>
     mermaid.initialize({
diff --git a/src/diagrams/state/shapes.js b/src/diagrams/state/shapes.js
index d8e1fdf5a..bed40cf47 100644
--- a/src/diagrams/state/shapes.js
+++ b/src/diagrams/state/shapes.js
@@ -206,7 +206,7 @@ const drawForkJoinState = (g, stateDef) => {
   let width = 70;
   let height = 7;
 
-  if (Math.PI > 3 || stateDef.parentId) {
+  if (stateDef.parentId) {
     let tmp = width;
     width = height;
     height = tmp;
@@ -296,6 +296,8 @@ export const drawNote = (text, g) => {
  * @param {*} elem
  * @param {*} stateDef
  */
+
+let cnt = 0;
 export const drawState = function(elem, stateDef, graph, doc) {
   const id = stateDef.id;
   const stateInfo = {
diff --git a/src/diagrams/state/stateDb.js b/src/diagrams/state/stateDb.js
index b84d66fad..11fc72c14 100644
--- a/src/diagrams/state/stateDb.js
+++ b/src/diagrams/state/stateDb.js
@@ -38,6 +38,7 @@ let currentDocument = documents.root;
 
 let startCnt = 0;
 let endCnt = 0;
+let stateCnt = 0;
 
 /**
  * Function called by parser when a node definition has been found.
diff --git a/src/diagrams/state/stateRenderer.js b/src/diagrams/state/stateRenderer.js
index 6f6c8e3c0..d1f91c237 100644
--- a/src/diagrams/state/stateRenderer.js
+++ b/src/diagrams/state/stateRenderer.js
@@ -94,7 +94,7 @@ export const draw = function(text, id) {
 
   diagram.attr('height', '100%');
   // diagram.attr('width', 'fit-content');
-  diagram.attr('style', `max-width: ${bounds.width * 1.5 + conf.padding * 2};`);
+  diagram.attr('style', `width: ${bounds.width * 3 + conf.padding * 2};`);
   diagram.attr(
     'viewBox',
     `${conf.padding * -1} ${conf.padding * -1} ` +
@@ -152,6 +152,7 @@ const renderDoc = (doc, diagram, parentId) => {
 
   total = keys.length;
   let first = true;
+
   for (let i = 0; i < keys.length; i++) {
     const stateDef = states[keys[i]];
 
@@ -227,11 +228,12 @@ const renderDoc = (doc, diagram, parentId) => {
   dagre.layout(graph);
 
   logger.debug('Graph after layout', graph.nodes());
+  const svgElem = diagram.node();
 
   graph.nodes().forEach(function(v) {
     if (typeof v !== 'undefined' && typeof graph.node(v) !== 'undefined') {
-      logger.debug('Node ' + v + ': ' + JSON.stringify(graph.node(v)));
-      d3.select('#' + v).attr(
+      logger.warn('Node ' + v + ': ' + JSON.stringify(graph.node(v)));
+      d3.select('#' + svgElem.id + ' #' + v).attr(
         'transform',
         'translate(' +
           (graph.node(v).x - graph.node(v).width / 2) +
@@ -241,15 +243,17 @@ const renderDoc = (doc, diagram, parentId) => {
             graph.node(v).height / 2) +
           ' )'
       );
-      d3.select('#' + v).attr('data-x-shift', graph.node(v).x - graph.node(v).width / 2);
-      const dividers = document.querySelectorAll('#' + v + ' .divider');
+      d3.select('#' + svgElem.id + ' #' + v).attr(
+        'data-x-shift',
+        graph.node(v).x - graph.node(v).width / 2
+      );
+      const dividers = document.querySelectorAll('#' + svgElem.id + ' #' + v + ' .divider');
       dividers.forEach(divider => {
         const parent = divider.parentElement;
         let pWidth = 0;
         let pShift = 0;
         if (parent) {
           if (parent.parentElement) pWidth = parent.parentElement.getBBox().width;
-
           pShift = parseInt(parent.getAttribute('data-x-shift'), 10);
           if (Number.isNaN(pShift)) {
             pShift = 0;
@@ -263,7 +267,7 @@ const renderDoc = (doc, diagram, parentId) => {
     }
   });
 
-  let stateBox = diagram.node().getBBox();
+  let stateBox = svgElem.getBBox();
 
   graph.edges().forEach(function(e) {
     if (typeof e !== 'undefined' && typeof graph.edge(e) !== 'undefined') {
@@ -272,7 +276,8 @@ const renderDoc = (doc, diagram, parentId) => {
     }
   });
 
-  stateBox = diagram.node().getBBox();
+  stateBox = svgElem.getBBox();
+  console.warn('Diagram node', svgElem.id);
   const stateInfo = {
     id: parentId ? parentId : 'root',
     label: parentId ? parentId : 'root',