@svgedit/svgcanvas 7.2.6 → 7.4.1

package/core/undo.js

@@ -46,11 +46,26 @@ export const getUndoManager = () => {
       if (eventType === EventTypes.BEFORE_UNAPPLY || eventType === EventTypes.BEFORE_APPLY) {
         svgCanvas.clearSelection()
       } else if (eventType === EventTypes.AFTER_APPLY || eventType === EventTypes.AFTER_UNAPPLY) {
+        const cmdType = cmd.type()
+        const isApply = (eventType === EventTypes.AFTER_APPLY)
+        if (cmdType === 'ChangeElementCommand' && cmd.elem === svgCanvas.getSvgContent()) {
+          const values = isApply ? cmd.newValues : cmd.oldValues
+          if (values.width !== null && values.width !== undefined && values.width !== '') {
+            const newContentW = Number(values.width)
+            if (Number.isFinite(newContentW) && newContentW > 0) {
+              svgCanvas.contentW = newContentW
+            }
+          }
+          if (values.height !== null && values.height !== undefined && values.height !== '') {
+            const newContentH = Number(values.height)
+            if (Number.isFinite(newContentH) && newContentH > 0) {
+              svgCanvas.contentH = newContentH
+            }
+          }
+        }
         const elems = cmd.elements()
         svgCanvas.pathActions.clear()
         svgCanvas.call('changed', elems)
-        const cmdType = cmd.type()
-        const isApply = (eventType === EventTypes.AFTER_APPLY)
         if (cmdType === 'MoveElementCommand') {
           const parent = isApply ? cmd.newParent : cmd.oldParent
           if (parent === svgCanvas.getSvgContent()) {
@@ -116,7 +131,7 @@ export const getUndoManager = () => {
 * @param {Element} elem - The (text) DOM element to clone
 * @returns {Element} Cloned element
 */
-export const ffClone = function (elem) {
+export const ffClone = (elem) => {
   if (!isGecko()) { return elem }
   const clone = elem.cloneNode(true)
   elem.before(clone)
@@ -213,7 +228,7 @@ export const changeSelectedAttributeNoUndoMethod = (attr, newValue, elems) => {
         elem = ffClone(elem)
       }
       // Timeout needed for Opera & Firefox
-      // codedread: it is now possible for this function to be called with elements
+      // codedread: it is now possible for this to be called with elements
       // that are not in the selectedElements array, we need to only request a
       // selector if the element is in that array
       if (selectedElements.includes(elem)) {
@@ -264,7 +279,7 @@ export const changeSelectedAttributeNoUndoMethod = (attr, newValue, elems) => {
 * @param {Element[]} elems - The DOM elements to apply the change to
 * @returns {void}
 */
-export const changeSelectedAttributeMethod = function (attr, val, elems) {
+export const changeSelectedAttributeMethod = (attr, val, elems) => {
   const selectedElements = svgCanvas.getSelectedElements()
   elems = elems || selectedElements
   svgCanvas.undoMgr.beginUndoableChange(attr, elems)

package/core/units.js

@@ -6,6 +6,8 @@
  * @copyright 2010 Alexis Deveria, 2010 Jeff Schiller
  */
 
+import { error } from '../common/logger.js'
+
 const NSSVG = 'http://www.w3.org/2000/svg'
 
 const wAttrs = ['x', 'x1', 'cx', 'rx', 'width']
@@ -62,7 +64,7 @@ let typeMap_ = {}
  * @param {module:units.ElementContainer} elementContainer - An object implementing the ElementContainer interface.
  * @returns {void}
  */
-export const init = function (elementContainer) {
+export const init = (elementContainer) => {
   elementContainer_ = elementContainer
 
   // Get correct em/ex values by creating a temporary SVG.
@@ -124,7 +126,7 @@ export const shortFloat = (val) => {
     return Number(Number(val).toFixed(digits))
   }
   if (Array.isArray(val)) {
-    return shortFloat(val[0]) + ',' + shortFloat(val[1])
+    return `${shortFloat(val[0])},${shortFloat(val[1])}`
   }
   return Number.parseFloat(val).toFixed(digits) - 0
 }
@@ -214,8 +216,8 @@ export const convertToNum = (attr, val) => {
     }
     return num * Math.sqrt((width * width) + (height * height)) / Math.sqrt(2)
   }
-  const unit = val.substr(-2)
-  const num = val.substr(0, val.length - 2)
+  const unit = val.slice(-2)
+  const num = val.slice(0, -2)
   // Note that this multiplication turns the string into a number
   return num * typeMap_[unit]
 }
@@ -237,7 +239,7 @@ export const isValidUnit = (attr, val, selectedElement) => {
     // Not a number, check if it has a valid unit
     val = val.toLowerCase()
     return Object.keys(typeMap_).some((unit) => {
-      const re = new RegExp('^-?[\\d\\.]+' + unit + '$')
+      const re = new RegExp(`^-?[\\d\\.]+${unit}$`)
       return re.test(val)
     })
   }
@@ -253,7 +255,7 @@ export const isValidUnit = (attr, val, selectedElement) => {
     try {
       const elem = elementContainer_.getElement(val)
       result = (!elem || elem === selectedElement)
-    } catch (e) { console.error(e) }
+    } catch (e) { error('Error getting element by ID', e, 'units') }
     return result
   }
   return true