Class Diagram: Adjust spacing of newly added lines to a class (#357)

ls1intum · Aug 9, 2024 · 38209a3 · 38209a3
1 parent b7efbbd
commit 38209a3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/src/main/packages/common/uml-classifier/uml-classifier-member.ts b/src/main/packages/common/uml-classifier/uml-classifier-member.ts
@@ -4,7 +4,7 @@ import { ILayoutable } from '../../../services/layouter/layoutable';
 import { IUMLElement, UMLElement } from '../../../services/uml-element/uml-element';
 import { UMLElementFeatures } from '../../../services/uml-element/uml-element-features';
 import { assign } from '../../../utils/fx/assign';
-import { IBoundary } from '../../../utils/geometry/boundary';
+import { IBoundary, computeDimension } from '../../../utils/geometry/boundary';
 import { Text } from '../../../utils/svg/text';
 
 export abstract class UMLClassifierMember extends UMLElement {
@@ -19,7 +19,7 @@ export abstract class UMLClassifierMember extends UMLElement {
     updatable: false,
   };
 
-  bounds: IBoundary = { ...this.bounds, height: 30 };
+  bounds: IBoundary = { ...this.bounds, height: computeDimension(1.0, 30) };
 
   constructor(values?: DeepPartial<IUMLElement>) {
     super(values);

diff --git a/src/main/packages/uml-class-diagram/class-preview.ts b/src/main/packages/uml-class-diagram/class-preview.ts
@@ -39,7 +39,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 0,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
 
@@ -50,7 +50,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 0,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
 
@@ -71,7 +71,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 40,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   const umlAbstractMethod = new UMLClassMethod({
@@ -81,7 +81,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 70,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   umlAbstract.ownedElements = [umlAbstractAttribute.id, umlAbstractMethod.id];
@@ -104,7 +104,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 50,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   const umlInterfaceMethod = new UMLClassMethod({
@@ -114,7 +114,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 80,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   umlInterface.ownedElements = [umlInterfaceAttribute.id, umlInterfaceMethod.id];
@@ -137,7 +137,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 50,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   const umlEnumerationCase2 = new UMLClassAttribute({
@@ -147,7 +147,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 80,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   const umlEnumerationCase3 = new UMLClassAttribute({
@@ -157,7 +157,7 @@ export const composeClassPreview: ComposePreview = (layer: ILayer, translate: (i
       x: 0,
       y: 110,
       width: computeDimension(1.0, 200),
-      height: computeDimension(1.0, 40),
+      height: computeDimension(1.0, 30),
     },
   });
   umlEnumeration.ownedElements = [umlEnumerationCase1.id, umlEnumerationCase2.id, umlEnumerationCase3.id];