decrease whitespace in ellipses having long text

chonton · mojoaxel · commit dd3639f2adf5 · 2016-08-11T10:32:56.000+02:00
diff --git a/lib/network/modules/components/nodes/shapes/Ellipse.js b/lib/network/modules/components/nodes/shapes/Ellipse.js
@@ -11,11 +11,8 @@ class Ellipse extends NodeBase {
     if (this.width === undefined) {
       var textSize = this.labelModule.getTextSize(ctx, selected);
 
-      this.width = textSize.width * 1.5;
       this.height = textSize.height * 2;
-      if (this.width < this.height) {
-        this.width = this.height;
-      }
+      this.width = textSize.width + this.height;
       this.radius = 0.5*this.width;
     }
   }
@@ -82,4 +79,4 @@ class Ellipse extends NodeBase {
   }
 }
 
-export default Ellipse;
+export default Ellipse;

Original file line number	Diff line number	Diff line change
`@@ -11,11 +11,8 @@ class Ellipse extends NodeBase {`
`11`	`11`	`if (this.width === undefined) {`
`12`	`12`	`var textSize = this.labelModule.getTextSize(ctx, selected);`
`13`	`13`
`14`		`- this.width = textSize.width * 1.5;`
`15`	`14`	`this.height = textSize.height * 2;`
`16`		`- if (this.width < this.height) {`
`17`		`- this.width = this.height;`
`18`		`- }`
	`15`	`+ this.width = textSize.width + this.height;`
`19`	`16`	`this.radius = 0.5*this.width;`
`20`	`17`	`}`
`21`	`18`	`}`
`@@ -82,4 +79,4 @@ class Ellipse extends NodeBase {`
`82`	`79`	`}`
`83`	`80`	`}`
`84`	`81`
`85`		`-export default Ellipse;`
	`82`	`+export default Ellipse;`