@teachinglab/omd 0.7.8 → 0.7.10

package/canvas/features/resizeHandleManager.js

@@ -445,13 +445,64 @@ export class ResizeHandleManager {
         if (!this.selectedElement) return { x: 0, y: 0, width: 0, height: 0 };
         
         try {
-            // Get bounding box of the content inside the wrapper
+            // Get the content inside the wrapper (usually an SVG element)
             const content = this.selectedElement.firstElementChild;
-            if (content) {
-                return content.getBBox();
-            } else {
+            if (!content) {
                 return this.selectedElement.getBBox();
             }
+            
+            // For elements with clip paths (like coordinate planes), 
+            // the graph lines extend beyond the visible area causing getBBox() to be too large.
+            // We need to find the clip rect to get the actual visible bounds.
+            const clipPaths = content.querySelectorAll('clipPath');
+            
+            if (clipPaths.length > 0) {
+                let maxArea = 0;
+                let clipBounds = null;
+                
+                for (const clipPath of clipPaths) {
+                    const rect = clipPath.querySelector('rect');
+                    if (rect) {
+                        const w = parseFloat(rect.getAttribute('width')) || 0;
+                        const h = parseFloat(rect.getAttribute('height')) || 0;
+                        const x = parseFloat(rect.getAttribute('x')) || 0;
+                        const y = parseFloat(rect.getAttribute('y')) || 0;
+                        
+                        const area = w * h;
+                        if (area > maxArea) {
+                            maxArea = area;
+                            
+                            // Find the transform on the content group
+                            const contentGroup = content.firstElementChild;
+                            let tx = 0, ty = 0;
+                            if (contentGroup) {
+                                const transform = contentGroup.getAttribute('transform');
+                                if (transform) {
+                                    const translateMatch = transform.match(/translate\(\s*([^,]+)(?:,\s*([^)]+))?\s*\)/);
+                                    if (translateMatch) {
+                                        tx = parseFloat(translateMatch[1]) || 0;
+                                        ty = parseFloat(translateMatch[2]) || 0;
+                                    }
+                                }
+                            }
+                            
+                            clipBounds = {
+                                x: x + tx,
+                                y: y + ty,
+                                width: w,
+                                height: h
+                            };
+                        }
+                    }
+                }
+                
+                if (clipBounds) {
+                    return clipBounds;
+                }
+            }
+            
+            // Fallback to getBBox if no clip path found
+            return content.getBBox();
         } catch (error) {
             // Fallback if getBBox fails
             return { x: 0, y: 0, width: 100, height: 100 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teachinglab/omd",
-  "version": "0.7.8",
+  "version": "0.7.10",
   "description": "omd",
   "main": "./index.js",
   "module": "./index.js",

package/src/omdTable.js

@@ -172,22 +172,46 @@ export class omdTable extends jsvgGroup
         this.data = [];
         this.headers = ['x', 'y'];
 
-        // Basic normalization for inline math
         let expression = this.equation;
-        if (expression.toLowerCase().startsWith('y=')) {
-            expression = expression.substring(2).trim();
+        // Remove "y=" from start or "=y" from end, case insensitive, handling spaces
+        expression = expression.replace(/^\s*y\s*=\s*/i, '').replace(/\s*=\s*y\s*$/i, '');
+
+        try {
+            // Use math.js if available (preferred)
+            if (typeof math !== 'undefined' && math.compile) {
+                const compiled = math.compile(expression);
+                for (let x = this.xMin; x <= this.xMax; x += this.stepSize) {
+                    try {
+                        let y = compiled.evaluate({ x });
+                        // Round to 2 decimal places for display
+                        y = Math.round(y * 100) / 100;
+                        this.data.push([x, y]);
+                    } catch (e) {
+                        // Skip points that fail to evaluate
+                    }
+                }
+                return;
+            }
+        } catch (e) {
+            console.warn("math.js evaluation failed, falling back to simple parser:", e);
         }
+
+        // Fallback to simple regex-based parsing
         expression = expression
             .replace(/(\d)([a-z])/gi, '$1*$2')
             .replace(/([a-z])(\d)/gi, '$1*$2')
             .replace(/\^/g, '**');
 
-        const evaluateExpression = new Function('x', `return ${expression};`);
+        try {
+            const evaluateExpression = new Function('x', `return ${expression};`);
 
-        for (let x = this.xMin; x <= this.xMax; x += this.stepSize) {
-            let y = evaluateExpression(x);
-            y = Math.round(y * 100) / 100;
-            this.data.push([x, y]);
+            for (let x = this.xMin; x <= this.xMax; x += this.stepSize) {
+                let y = evaluateExpression(x);
+                y = Math.round(y * 100) / 100;
+                this.data.push([x, y]);
+            }
+        } catch (e) {
+            console.error("Error generating table data from equation:", e);
         }
     }