@vectoriox/iox-builder 1.9.0 → 1.9.2

package/fesm2022/vectoriox-iox-builder.mjs

@@ -2239,7 +2239,11 @@ class TypographyGroupStyleConfig extends GroupStyleConfig {
             new TraitConfig('fontSize', 'Font Size', TraitInputType.Scrub, { min: 8, max: 200, step: 1, units: UNITS_ALL }, '16px'),
             new TraitConfig('fontWeight', 'Weight', TraitInputType.Scrub, { min: 100, max: 900, step: 100, units: [''] }, '400'),
             new TraitConfig('color', 'Color', TraitInputType.Color, { allowGradient: false, allowTransparent: false, formats: ['hex', 'rgb', 'hsl'] }, '#000000'),
-            new TraitConfig('lineHeight', 'Line Height', TraitInputType.Scrub, { min: 0.5, max: 5, step: 0.05, units: [''] }, '1.5'),
+            // Unitless on purpose (`units: ['']`): a unitless line-height inherits as a RATIO — each
+            // descendant multiplies it by its OWN font-size — whereas `1.5em`/`24px` inherits a fixed
+            // computed length that ignores the child's font-size. min:0 so a line box can be collapsed
+            // completely (icons/glyph-only elements), which `0.5` used to block.
+            new TraitConfig('lineHeight', 'Line Height', TraitInputType.Scrub, { min: 0, max: 5, step: 0.05, units: [''] }, '1.5'),
             new TraitConfig('letterSpacing', 'Spacing', TraitInputType.Scrub, { min: -5, max: 50, step: 0.5, units: UNITS_FIXED }, '0px'),
             new TraitConfig('textAlign', '', TraitInputType.SelectButton, [
                 { value: 'left', icon: 'ph-thin ph-text-align-left' },
@@ -4250,6 +4254,19 @@ class BuilderComponent {
         this.panOriginY = 0;
         this.hoverHideRaf = 0;
         this.canvasContentHeight = 0;
+        /**
+         * Cancel the browser's default navigation for any link clicked on the editing canvas.
+         *
+         * Bound property (not a method) so add/removeEventListener share one reference. Matches the
+         * clicked target's nearest `a[href]`, so it covers a Link/Button host as well as a click on
+         * anything nested inside a LinkedContainer. Only the DEFAULT action is cancelled — propagation
+         * is untouched, so node selection, the overlay and the context menu all behave normally.
+         */
+        this.blockCanvasNavigation = (event) => {
+            const target = event.target;
+            if (target?.closest?.('a[href]'))
+                event.preventDefault();
+        };
         this._scrollbarTimer = null;
     }
     ngOnInit() {
@@ -4325,6 +4342,16 @@ class BuilderComponent {
                 }
             });
             this.canvasRo.observe(canvasEl);
+            // The canvas is an EDITING surface: clicking a Link / LinkedContainer / Button (anything
+            // rendering an <a href>) must select it, never navigate — a redirect unloads the builder
+            // and loses unsaved work. Capture phase so it runs before the node's own click handler,
+            // and canvas-level so it covers EVERY anchor, including repeater/slider preview clones
+            // (RenderDirective skips its per-node listeners for those) and any future link component.
+            // preventDefault only cancels the browser's default navigation — the click still
+            // propagates, so selection/overlay behaviour is unchanged.
+            // `auxclick` covers middle-click, which opens a tab without firing `click`.
+            canvasEl.addEventListener('click', this.blockCanvasNavigation, true);
+            canvasEl.addEventListener('auxclick', this.blockCanvasNavigation, true);
         }
         this.initLenis();
         // Moveable path (flag-gated — interact.js is untouched when off). Drive drag for the selected
@@ -4354,6 +4381,8 @@ class BuilderComponent {
         const canvasEl = document.getElementById('canvas-preview');
         if (canvasEl) {
             this.dragEngine.unregisterDropzone('canvas-preview', canvasEl);
+            canvasEl.removeEventListener('click', this.blockCanvasNavigation, true);
+            canvasEl.removeEventListener('auxclick', this.blockCanvasNavigation, true);
         }
     }
     /**
@@ -7491,10 +7520,10 @@ class BuilderIconComponentConfig extends ComponentConfig {
             fontSize: '24px',
             color: '#374151',
             // An icon is a glyph, not a paragraph — the Typography group's default line-height of
-            // 1.5 adds phantom vertical space and misaligns it. Default it to 1 so the box hugs the
-            // glyph. Serialized into styleProps (self-describing node), so the builder AND the SSR
-            // engine both compile `line-height: 1` — no engine-side special-case needed.
-            lineHeight: '1',
+            // 1.5 adds phantom vertical space and misaligns it. Default to 0 so the line box is fully
+            // collapsed and the icon contributes no extra vertical space (spacing comes from
+            // padding/margin instead). Serialized into styleProps (self-describing node).
+            lineHeight: '0',
         });
     }
 }