refactor: extract shared commitHeightTransform helper for 1D barcodes

Author: u8array

src/registry/barcode1d.tsx

@@ -2,6 +2,7 @@ import type { ObjectTypeDefinition, ObjectGroup, LabelObjectBase } from '../type
 import { useT } from '../lib/useT';
 import { inputCls, labelCls } from '../components/Properties/styles';
 import { fieldPos } from './zplHelpers';
+import { commitHeightTransform } from './transformHelpers';
 
 export interface Barcode1DProps {
   content: string;
@@ -57,11 +58,7 @@ export function createBarcode1D(config: Barcode1DConfig): ObjectTypeDefinition<B
     defaultSize: { width: 300, height: 120 },
     heightLocked: config.heightLocked,
 
-    commitTransform: config.heightLocked
-      ? undefined
-      : (obj, { sy, snap }) => ({
-          height: Math.max(1, snap(Math.round(obj.props.height * sy))),
-        }),
+    commitTransform: config.heightLocked ? undefined : commitHeightTransform,
 
     toZPL: (obj: LabelObjectBase & { props: Barcode1DProps }) => {
       const p = obj.props;

src/registry/code128.tsx

@@ -2,6 +2,7 @@ import type { ObjectTypeDefinition } from '../types/ObjectType';
 import { useT } from '../lib/useT';
 import { inputCls, labelCls } from '../components/Properties/styles';
 import { fieldPos } from './zplHelpers';
+import { commitHeightTransform } from './transformHelpers';
 
 export interface Code128Props {
   content: string;
@@ -24,9 +25,7 @@ export const code128: ObjectTypeDefinition<Code128Props> = {
   },
   defaultSize: { width: 300, height: 120 },
 
-  commitTransform: (obj, { sy, snap }) => ({
-    height: Math.max(1, snap(Math.round(obj.props.height * sy))),
-  }),
+  commitTransform: commitHeightTransform,
 
   toZPL: (obj) => {
     const p = obj.props;

src/registry/code39.tsx

@@ -2,6 +2,7 @@ import type { ObjectTypeDefinition } from '../types/ObjectType';
 import { useT } from '../lib/useT';
 import { inputCls, labelCls } from '../components/Properties/styles';
 import { fieldPos } from './zplHelpers';
+import { commitHeightTransform } from './transformHelpers';
 
 export interface Code39Props {
   content: string;
@@ -24,9 +25,7 @@ export const code39: ObjectTypeDefinition<Code39Props> = {
   },
   defaultSize: { width: 300, height: 120 },
 
-  commitTransform: (obj, { sy, snap }) => ({
-    height: Math.max(1, snap(Math.round(obj.props.height * sy))),
-  }),
+  commitTransform: commitHeightTransform,
 
   toZPL: (obj) => {
     const p = obj.props;

src/registry/ean13.tsx

@@ -2,6 +2,7 @@ import type { ObjectTypeDefinition } from '../types/ObjectType';
 import { useT } from '../lib/useT';
 import { inputCls, labelCls } from '../components/Properties/styles';
 import { fieldPos } from './zplHelpers';
+import { commitHeightTransform } from './transformHelpers';
 
 export interface Ean13Props {
   content: string;        // 12 digits — ZPL appends the check digit automatically
@@ -22,9 +23,7 @@ export const ean13: ObjectTypeDefinition<Ean13Props> = {
   },
   defaultSize: { width: 300, height: 120 },
 
-  commitTransform: (obj, { sy, snap }) => ({
-    height: Math.max(1, snap(Math.round(obj.props.height * sy))),
-  }),
+  commitTransform: commitHeightTransform,
 
   toZPL: (obj) => {
     const p = obj.props;

src/registry/transformHelpers.ts

@@ -22,6 +22,18 @@ export function commitWidthHeightTransform<P extends WidthHeightProps>(
   } as Partial<P>;
 }
 
+/** Shared commitTransform for 1D barcodes — only the bar height scales (width
+ *  is determined by content + module width, not by the resize anchor). */
+export function commitHeightTransform<P extends { height: number }>(
+  obj: LabelObjectBase & { props: P },
+  ctx: TransformContext,
+): Partial<P> {
+  const { sy, snap } = ctx;
+  return {
+    height: Math.max(1, snap(Math.round(obj.props.height * sy))),
+  } as Partial<P>;
+}
+
 interface Stacked2DProps {
   rowHeight: number;
   moduleWidth: number;