Merge pull request #1 from adrai/master

update

README.md

@@ -5,7 +5,7 @@
 [example](https://github.com/adrai/flowchart.js/blob/master/example/index.html)
 
 #Requirements
-You will need [Raphaël](http://raphaeljs.com/)
+You will need [Raphaël](http://www.raphaeljs.com/)
 
 #Usage
 

package.json

@@ -1,7 +1,7 @@
 {
   "author": "adrai",
   "name": "flowchart.js",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "main": "./index",
   "private": false,
   "engines": {

releasenotes.md

@@ -1,3 +1,11 @@
+### v1.6.3
+
+- Allow going to same symbol thanks to [nonylene](https://github.com/nonylene) [##83](https://github.com/adrai/flowchart.js/pull/#83)
+
+### v1.6.2
+
+- Fixed not calculate viewBox and size properly thanks to [jackycute](https://github.com/jackycute) [##74](https://github.com/adrai/flowchart.js/issues/#74)
+
 ### v1.6.1
 
 - Fixed lines are not included in the calculation of viewBox and size thanks to [jackycute](https://github.com/jackycute) [##72](https://github.com/adrai/flowchart.js/issues/#72) [##67](https://github.com/adrai/flowchart.js/issues/#67)

src/flowchart.chart.js

@@ -60,6 +60,8 @@ FlowChart.prototype.render = function() {
       len = 0,
       maxX = 0,
       maxY = 0,
+      minX = 0,
+      minY = 0,
       symbol,
       line;
 
@@ -103,23 +105,38 @@ FlowChart.prototype.render = function() {
       maxY = y;
     }
   }
-    
+
   for (i = 0, len = this.lines.length; i < len; i++) {
     line = this.lines[i].getBBox();
-    var x = line.x2;
-    var y = line.y2;
-    if (x > maxX) {
-      maxX = x;
+    var x = line.x;
+    var y = line.y;
+    var x2 = line.x2;
+    var y2 = line.y2;
+    if (x < minX) {
+      minX = x;
     }
-    if (y > maxY) {
-      maxY = y;
+    if (y < minY) {
+      minY = y;
+    }
+    if (x2 > maxX) {
+      maxX = x2;
+    }
+    if (y2 > maxY) {
+      maxY = y2;
     }
   }
 
   var scale = this.options['scale'];
   var lineWidth = this.options['line-width'];
-  this.paper.setSize((maxX * scale) + (lineWidth * scale), (maxY * scale) + (lineWidth * scale));
-  this.paper.setViewBox(0, 0, maxX + lineWidth, maxY + lineWidth, true);
+
+  if (minX < 0) minX -= lineWidth;
+  if (minY < 0) minY -= lineWidth;
+
+  var width = maxX + lineWidth - minX;
+  var height = maxY + lineWidth - minY;
+
+  this.paper.setSize(width * scale, height * scale);
+  this.paper.setViewBox(minX, minY, width, height, true);
 };
 
 FlowChart.prototype.clean = function() {

src/flowchart.symbol.js

@@ -239,7 +239,7 @@ Symbol.prototype.drawLineTo = function(symbol, text, origin) {
   var isOnSameColumn = x === symbolX,
       isOnSameLine = y === symbolY,
       isUnder = y < symbolY,
-      isUpper = y > symbolY,
+      isUpper = y > symbolY || this === symbol,
       isLeft = x > symbolX,
       isRight = x < symbolX;