Merge pull request #300 from LarryKlugerDS/master

Issue 299: Sequence diagram Loops: changing boxMargin spoils the "loop" notation

CONTRIBUTING.md

@@ -53,20 +53,24 @@ Fork, then:
 npm install
 ```
 
-Then the dependencies will have been installed. You use gulp as build tool.
+Then the dependencies will have been installed. You use gulp and npm calls as build tools.
 
 The following targets are probably interesting:
 
 * jison - compiles the jison grammars to parser files
-* dist - complies files to the dist catalog
 
 for instance:
 ```
-gulp jasmine
+gulp jison
 ```
 To run the tests:
 ```
 npm run karma
 ```
 
+To build the /dist directory
+```
+npm run dist
+```
+
 Thanks, Knut Sveidqvist

src/diagrams/sequenceDiagram/svgDraw.js

@@ -151,8 +151,8 @@ exports.drawLoop = function(elem,bounds,labelText, conf){
  txt.text = labelText;
  txt.x = bounds.startx;
  txt.y = bounds.starty;
- txt.labelMargin = 1.5 * conf.boxMargin;
- txt.class = 'labelText';
+ txt.labelMargin = 1.5 * 10; // This is the small box that says "loop"
+ txt.class = 'labelText'; // Its size & position are fixed.
  txt.fill = 'white';
 
  exports.drawLabel(g,txt);