fix: move adjustSvgDimension in do actions

- Ensured that `adjustSvgDimension` is called after drawing complete
- Added 100px offset to y dimension

refs: #277 (comment)

Author: yinanazhou

src/js/app.js

@@ -15,6 +15,7 @@ import { debug } from './conf'
 
 import newApp from './new/app'
 import { downloadAs } from './new/utils/file'
+import { adjustSvgDimensions } from './ui'
 
 // Clicking selects, exposed globally
 window.selected = []
@@ -200,6 +201,8 @@ export function do_relation(type, id, redoing = false) {
 
   // Update hierarchy tree if visible
   window.relationTreeInstance?.updateIfVisible()
+
+  adjustSvgDimensions(getCurrentDrawContext())
 }
 
 export function do_comborelation(type) {
@@ -253,6 +256,8 @@ export function do_metarelation(type, id, redoing = false) {
 
   // Update hierarchy tree if visible
   window.relationTreeInstance?.updateIfVisible()
+
+  adjustSvgDimensions(getCurrentDrawContext())
 }
 
 var rerendered_after_action
@@ -634,6 +639,7 @@ function finalize_draw_context(new_draw_context) {
   console.groupEnd()
   draw_graph(new_draw_context)
   setCurrentDrawContext(new_draw_context)
+  adjustSvgDimensions(new_draw_context)
 }
 
 function render_mei(mei) {

src/js/draw.js

@@ -115,9 +115,6 @@ export function draw_relation(draw_context, mei_graph, g_elem) {
   // Remember what we're adding
   added.push(group)
 
-  // Adjust SVG dimensions if needed
-  adjustSvgDimensions(draw_context)
-
   return added
 }
 
@@ -426,8 +423,5 @@ export function draw_metarelation(draw_context, mei_graph, g_elem) {
   // Remember what we're adding
   added.push(g_elem)
 
-  // Adjust SVG dimensions if needed
-  adjustSvgDimensions(draw_context)
-
   return added
 }

src/js/ui.js

@@ -555,6 +555,7 @@ export const setCurrentDrawContext = drawContext => {
 }
 
 export function adjustSvgDimensions(draw_context) {
+  console.trace('adjustSvgDimensions')
   const svg_elem = draw_context.svg_elem
   const rootSvg = svg_elem.getElementsByTagName('svg')[0]
   const definitionScale = svg_elem.getElementsByClassName('definition-scale')[0]
@@ -611,5 +612,5 @@ export function adjustSvgDimensions(draw_context) {
 
   // Set the width and height on the container SVG in pixels
   rootSvg.setAttribute('width', `${newPixelWidth}px`)
-  rootSvg.setAttribute('height', `${newPixelHeight}px`)
+  rootSvg.setAttribute('height', `${newPixelHeight + 100}px`)
 }