remove: text decoration (#564)

b310-digital · Nov 14, 2024 · b87f9bb · b87f9bb
1 parent 9163619
commit b87f9bb
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 33 deletions.
diff --git a/teammapper-frontend/mmp/src/map/handlers/draw.ts b/teammapper-frontend/mmp/src/map/handlers/draw.ts
@@ -506,7 +506,6 @@ export default class Draw {
         div.style.setProperty('color', DOMPurify.sanitize(node.colors.name))
         div.style.setProperty('font-style', DOMPurify.sanitize(node.font.style))
         div.style.setProperty('font-weight', DOMPurify.sanitize(node.font.weight))
-        div.style.setProperty('text-decoration', DOMPurify.sanitize(node.font.decoration))
 
         div.style.setProperty('touch-action', 'none')
         div.style.setProperty('display', 'inline-block')

diff --git a/teammapper-frontend/mmp/src/map/handlers/nodes.ts b/teammapper-frontend/mmp/src/map/handlers/nodes.ts
@@ -348,9 +348,6 @@ export default class Nodes {
             case 'fontWeight':
                 updated = this.updateNodeFontWeight(node, value)
                 break
-            case 'textDecoration':
-                updated = this.updateNodeTextDecoration(node, value)
-                break
             case 'fontStyle':
                 updated = this.updateNodeFontStyle(node, value)
                 break
@@ -1025,29 +1022,6 @@ export default class Nodes {
         }
     }
 
-    /**
-     * Update the node text decoration.
-     * @param {Node} node
-     * @param {string} decoration
-     * @returns {boolean}
-     */
-    private updateNodeTextDecoration = (node: Node, decoration: string): boolean => {
-        if (decoration && typeof decoration !== 'string') {
-            Log.error('The text decoration must be a string', 'type')
-        }
-
-        if (node.font.decoration !== decoration) {
-            node.getNameDOM().style['text-decoration'] = DOMPurify.sanitize(decoration)
-
-            this.map.draw.updateNodeShapes(node)
-
-            node.font.decoration = decoration
-            return true
-        } else {
-            return false
-        }
-    }
-
     /**
      * Update the node locked status.
      * @param {Node} node
@@ -1140,7 +1114,6 @@ export const PropertyMapping = {
     backgroundColor: ['colors', 'background'],
     branchColor: ['colors', 'branch'],
     fontWeight: ['font', 'weight'],
-    textDecoration: [],
     fontStyle: ['font', 'style'],
     fontSize: ['font', 'size'],
     nameColor: ['colors', 'name'],

diff --git a/teammapper-frontend/mmp/src/map/models/node.ts b/teammapper-frontend/mmp/src/map/models/node.ts
@@ -163,5 +163,4 @@ export interface Font {
     size: number
     style: string
     weight: string
-    decoration: string
 }
diff --git a/teammapper-frontend/mmp/src/map/options.ts b/teammapper-frontend/mmp/src/map/options.ts
@@ -203,8 +203,7 @@ export const DefaultNodeValues: DefaultNodeProperties = {
     font: {
         size: 16,
         style: 'normal',
-        weight: 'normal',
-        decoration: ''
+        weight: 'normal'
     },
     locked: true,
     detached: false,
@@ -233,8 +232,7 @@ export const DefaultRootNodeValues: DefaultNodeProperties = {
     font: {
         size: 20,
         style: 'normal',
-        weight: 'normal',
-        decoration: ''
+        weight: 'normal'
     },
     locked: true,
     detached: false,