@designcrowd/library.brand-page 5.18.9-ast-padding-2 → 5.18.9

package/src/brand-page-maker/convertors/contact-layout-style.convertor.ts

@@ -6,7 +6,6 @@ import {
   ALIGN_CARD_CENTER_IMAGE,
   TEXT_ONLY,
   ALIGN_CARD_CENTER,
-  CONTACT_FORM_LAYOUT_DEFAULT_PADDING,
 } from '@brand-page/constants';
 import type { ContactFormContent } from '@brand-page';
 
@@ -85,15 +84,8 @@ export const contactLayoutStyleConvertor: ValueConvertor<
       textAlign = 'left';
     }
 
-    // Switching layout adopts that layout's canonical look — same as the
-    // textAlign resets below — including its default block padding.
-    const padding =
-      CONTACT_FORM_LAYOUT_DEFAULT_PADDING[selectedOption?.value ?? ''] ?? 0;
-
     return {
       layoutStyle: selectedOption?.value,
-      topPadding: padding,
-      bottomPadding: padding,
       titleStyle: {
         textAlign,
       },

package/src/brand-page-maker/models/content-settings.ts

@@ -24,8 +24,6 @@ import {
   EMPTY_LOGO_IMAGE_MODEL,
   BRAND_PAGE_TYPES,
   SENSITIVE_CONTENT_TYPES,
-  CONTACT_FORM_LAYOUT_DEFAULT_PADDING,
-  TESTIMONIAL_DEFAULT_PADDING,
 } from '@brand-page';
 import type {
   ButtonContent,
@@ -173,8 +171,6 @@ export const createInitialImageGalleryContent = (): ImageGalleryContent => {
     galleryStyles: {
       galleryType: 'grid',
     },
-    topPadding: 0,
-    bottomPadding: 0,
   };
 };
 
@@ -186,8 +182,6 @@ export const createInitialVideoContent = (): VideoContent => {
     displayStyle: DISPLAY_STYLE_PREVIEW,
     mute: false,
     videoUrl: '',
-    topPadding: 0,
-    bottomPadding: 0,
   };
 };
 
@@ -245,10 +239,6 @@ export const createInitialTextContent = (
           ]
         : undefined,
     type: 'text',
-    // Brand Site text blocks sit in a 12px outer padding; Brand Page / Brand Contact use 0.
-    topPadding: options?.brandPageType === BRAND_PAGE_TYPES.brandSite ? 12 : 0,
-    bottomPadding:
-      options?.brandPageType === BRAND_PAGE_TYPES.brandSite ? 12 : 0,
     textSize: {
       fontSize: FONT_SIZE_BODY,
       lineHeight: LINE_HEIGHT_BODY,
@@ -292,8 +282,6 @@ export const createInitialMapContent = (): MapContent => {
       formattedAddress: '',
       placeId: '',
     },
-    topPadding: 0,
-    bottomPadding: 0,
   };
 };
 
@@ -303,8 +291,6 @@ export const createInitialMailingListContent = (
   return {
     id: uuid.v4(),
     type: 'mailing-list',
-    topPadding: 0,
-    bottomPadding: 0,
     title: contentSettingsTr(`mailingListTitle`),
     buttonText: contentSettingsTr(`subscribePlaceholder`),
     acceptedFields: 'emailOnly',
@@ -338,8 +324,6 @@ export const createInitialRequestPaymentContent = (
     successMessage: '',
     title: contentSettingsTr(`buyNowTitle`),
     type: 'request-payment',
-    topPadding: 0,
-    bottomPadding: 0,
     fillColor: {},
     defaultBorderColor: {},
     selectedBorderColor: {},
@@ -372,8 +356,6 @@ export const createInitialShopContent = (
   return {
     id: uuid.v4(),
     description: '',
-    topPadding: 16,
-    bottomPadding: 16,
     products: [],
     displayStyle: PAYMENT_OPTION_DISPLAY_TYPES.thumbnail,
     aspectRatio: PAYMENT_OPTION_IMAGE_TYPES.auto,
@@ -433,8 +415,6 @@ export const createInitialSupportMeContent = (
     successMessage: contentSettingsTr(`successSupportPlaceholder`),
     description: contentSettingsTr(`makeADonationDescription`),
     type: 'support-me',
-    topPadding: 0,
-    bottomPadding: 0,
     titleStyle: {
       applyAutoScale: options?.brandPageType === BRAND_PAGE_TYPES.brandSite,
       textStyleType: TEXT_STYLE_TYPES.headings,
@@ -473,21 +453,14 @@ export const createInitialContactFormContent: CreateInitialContentFunction<
     console.error('userState.emailAddress should not be undefined');
   }
 
-  // New blocks inherit the padding their initial layout previously had
-  // hard-coded — same source as the migration backfill for existing pages.
-  const layoutStyle =
-    options?.brandPageType === 'brandSite' ? 'center-card-aligned' : '';
-  const padding = CONTACT_FORM_LAYOUT_DEFAULT_PADDING[layoutStyle] ?? 0;
-
   return {
     id: uuid.v4(),
     title: contentSettingsTr(`contactUsTitle`),
     description: '',
     buttonText: contentSettingsTr(`sendPlaceholder`),
     displayStyle: DISPLAY_STYLE_PREVIEW,
-    layoutStyle,
-    topPadding: padding,
-    bottomPadding: padding,
+    layoutStyle:
+      options?.brandPageType === 'brandSite' ? 'center-card-aligned' : '',
     imageUrl: '',
     fields: [
       {
@@ -580,8 +553,6 @@ export const createInitialBannerSectionContent: CreateInitialContentFunction<
 > = (options?: CreateInitialContentOptions) => {
   return {
     id: uuid.v4(),
-    topPadding: 32,
-    bottomPadding: 32,
     imageUrl: '',
     title: contentSettingsTr(`thisIsABannerTitle`),
     description: '',
@@ -628,8 +599,6 @@ export const createInitialBookingSectionContent: CreateInitialContentFunction<
     buttonText: contentSettingsTr(`bookNowLabel`),
     url: '',
     type: 'booking-section',
-    topPadding: 32,
-    bottomPadding: 32,
     descriptionStyle: {
       applyAutoScale: options?.brandPageType === BRAND_PAGE_TYPES.brandSite,
       textStyleType: TEXT_STYLE_TYPES.paragraphs,
@@ -659,8 +628,6 @@ export const createInitialImagesSectionContent: CreateInitialContentFunction<
     title: contentSettingsTr(`imagesTitle`),
     description: '',
     type: 'images-section',
-    topPadding: 32,
-    bottomPadding: 32,
     displayStyle: 'centered',
     aspectRatio: 'Default',
     areImagesFullWidth: false,
@@ -716,8 +683,6 @@ export const createInitialTestimonialSectionContent: CreateInitialContentFunctio
     title: contentSettingsTr(`testimonialsTitle`),
     type: 'testimonial-section',
     displayStyle: 'centered',
-    topPadding: TESTIMONIAL_DEFAULT_PADDING,
-    bottomPadding: TESTIMONIAL_DEFAULT_PADDING,
     items: [],
     titleStyle: {
       applyAutoScale: options?.brandPageType === BRAND_PAGE_TYPES.brandSite,
@@ -766,8 +731,6 @@ export const createInitialAboutSectionContent: CreateInitialContentFunction<
       textAlign: 'center',
     },
     backgroundStyle: {},
-    topPadding: 32,
-    bottomPadding: 32,
   };
 };
 
@@ -830,8 +793,6 @@ export const createInitialSocialLinksContent = (
   return {
     id: uuid.v4(),
     type: 'social-links',
-    topPadding: 32,
-    bottomPadding: 32,
     title: contentSettingsTr(`followUsOnTitle`),
     links: [
       {
@@ -885,8 +846,6 @@ export const createInitialShapesSectionContent: CreateInitialContentFunction<
     title: contentSettingsTr(`shapesTitle`),
     description: '',
     type: 'shapes-section',
-    topPadding: 32,
-    bottomPadding: 32,
     displayStyle: 'centered',
     items: [
       {

package/src/bundles/bundled-translations.de-DE.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Farbe der Symbolfüllung",
     "iconBackgroundColorTitle": "Symbolhintergrundfarbe",
     "paddingTitle": "Polsterung",
-    "topPaddingTitle": "Polsterung oben",
-    "bottomPaddingTitle": "Polsterung unten",
     "dividerStyleTitle": "Trennlinienstil",
     "dividerColorTitle": "Trennlinienfarbe",
     "addItemTitle": "Artikel hinzufügen",

package/src/bundles/bundled-translations.es-ES.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Color de relleno del icono",
     "iconBackgroundColorTitle": "Color de fondo del icono",
     "paddingTitle": "Relleno",
-    "topPaddingTitle": "Relleno superior",
-    "bottomPaddingTitle": "Relleno inferior",
     "dividerStyleTitle": "Estilo del divisor",
     "dividerColorTitle": "Color del separador",
     "addItemTitle": "Añadir artículo",

package/src/bundles/bundled-translations.fr-CA.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Couleur de remplissage de l’icône",
     "iconBackgroundColorTitle": "Couleur d’arrière-plan de l’icône",
     "paddingTitle": "Marges intérieures",
-    "topPaddingTitle": "Marge intérieure supérieure",
-    "bottomPaddingTitle": "Marges intérieures inférieures",
     "dividerStyleTitle": "Style de séparateur",
     "dividerColorTitle": "Couleur du séparateur",
     "addItemTitle": "Ajouter un article",

package/src/bundles/bundled-translations.fr-FR.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Couleur de remplissage de l’icône",
     "iconBackgroundColorTitle": "Couleur d’arrière-plan de l’icône",
     "paddingTitle": "Marges intérieures",
-    "topPaddingTitle": "Marges intérieures supérieures",
-    "bottomPaddingTitle": "Marges intérieures inférieures",
     "dividerStyleTitle": "Style de séparateur",
     "dividerColorTitle": "Couleur du séparateur",
     "addItemTitle": "Ajouter un article",

package/src/bundles/bundled-translations.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Icon fill color",
     "iconBackgroundColorTitle": "Icon background color",
     "paddingTitle": "Padding",
-    "topPaddingTitle": "Top padding",
-    "bottomPaddingTitle": "Bottom padding",
     "dividerStyleTitle": "Divider Style",
     "dividerColorTitle": "Divider color",
     "addItemTitle": "Add Item",

package/src/bundles/bundled-translations.pt-BR.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Cor de preenchimento do ícone",
     "iconBackgroundColorTitle": "Cor de fundo do ícone",
     "paddingTitle": "Preenchimento",
-    "topPaddingTitle": "Preenchimento superior",
-    "bottomPaddingTitle": "Preenchimento inferior",
     "dividerStyleTitle": "Estilo do separador",
     "dividerColorTitle": "Cor do separador",
     "addItemTitle": "Adicionar item",

package/src/bundles/bundled-translations.pt-PT.json

@@ -1529,8 +1529,6 @@
     "iconFillColorTitle": "Cor de preenchimento do ícone",
     "iconBackgroundColorTitle": "Cor de fundo do ícone",
     "paddingTitle": "Preenchimento",
-    "topPaddingTitle": "Preenchimento superior",
-    "bottomPaddingTitle": "Preenchimento inferior",
     "dividerStyleTitle": "Estilo do separador",
     "dividerColorTitle": "Cor do separador",
     "addItemTitle": "Adicionar Item",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.de-DE.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Farbe der Symbolfüllung",
     "iconBackgroundColorTitle": "Symbolhintergrundfarbe",
     "paddingTitle": "Polsterung",
-    "topPaddingTitle": "Polsterung oben",
-    "bottomPaddingTitle": "Polsterung unten",
     "dividerStyleTitle": "Trennlinienstil",
     "dividerColorTitle": "Trennlinienfarbe",
     "addItemTitle": "Artikel hinzufügen",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.es-ES.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Color de relleno del icono",
     "iconBackgroundColorTitle": "Color de fondo del icono",
     "paddingTitle": "Relleno",
-    "topPaddingTitle": "Relleno superior",
-    "bottomPaddingTitle": "Relleno inferior",
     "dividerStyleTitle": "Estilo del divisor",
     "dividerColorTitle": "Color del separador",
     "addItemTitle": "Añadir artículo",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.fr-CA.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Couleur de remplissage de l’icône",
     "iconBackgroundColorTitle": "Couleur d’arrière-plan de l’icône",
     "paddingTitle": "Marges intérieures",
-    "topPaddingTitle": "Marge intérieure supérieure",
-    "bottomPaddingTitle": "Marges intérieures inférieures",
     "dividerStyleTitle": "Style de séparateur",
     "dividerColorTitle": "Couleur du séparateur",
     "addItemTitle": "Ajouter un article",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.fr-FR.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Couleur de remplissage de l’icône",
     "iconBackgroundColorTitle": "Couleur d’arrière-plan de l’icône",
     "paddingTitle": "Marges intérieures",
-    "topPaddingTitle": "Marges intérieures supérieures",
-    "bottomPaddingTitle": "Marges intérieures inférieures",
     "dividerStyleTitle": "Style de séparateur",
     "dividerColorTitle": "Couleur du séparateur",
     "addItemTitle": "Ajouter un article",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Icon fill color",
     "iconBackgroundColorTitle": "Icon background color",
     "paddingTitle": "Padding",
-    "topPaddingTitle": "Top padding",
-    "bottomPaddingTitle": "Bottom padding",
     "dividerStyleTitle": "Divider Style",
     "dividerColorTitle": "Divider color",
     "addItemTitle": "Add Item",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.pt-BR.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Cor de preenchimento do ícone",
     "iconBackgroundColorTitle": "Cor de fundo do ícone",
     "paddingTitle": "Preenchimento",
-    "topPaddingTitle": "Preenchimento superior",
-    "bottomPaddingTitle": "Preenchimento inferior",
     "dividerStyleTitle": "Estilo do separador",
     "dividerColorTitle": "Cor do separador",
     "addItemTitle": "Adicionar item",

package/src/shared/content-settings/components/content-settings/i18n/content-settings.pt-PT.json

@@ -113,8 +113,6 @@
     "iconFillColorTitle": "Cor de preenchimento do ícone",
     "iconBackgroundColorTitle": "Cor de fundo do ícone",
     "paddingTitle": "Preenchimento",
-    "topPaddingTitle": "Preenchimento superior",
-    "bottomPaddingTitle": "Preenchimento inferior",
     "dividerStyleTitle": "Estilo do separador",
     "dividerColorTitle": "Cor do separador",
     "addItemTitle": "Adicionar Item",

package/src/shared/content-settings/components/content-settings-item/ContentSettingsItem.vue

@@ -56,7 +56,7 @@
     >
       <li
         v-for="(field, index) in fields"
-        :key="`content-settings-item-field-${id}-${field.name}`"
+        :key="`content-settings-item-field-${id}-${index}`"
         class="tw-cursor-pointer"
         :class="{
           'tw-mb-3': index !== fields.length - 1,

package/src/shared/content-settings/components/focused-content-item-settings/FocusedContentItemSettings.vue

@@ -25,8 +25,8 @@
     >
       <ul>
         <li
-          v-for="field in group.fields"
-          :key="`content-settings-item-group-field-${id}-${group.id}-${field.name}`"
+          v-for="(field, fieldIndex) in group.fields"
+          :key="`content-settings-item-group-field-${id}-${group.id}-${fieldIndex}`"
         >
           <ContentSettingsItemField
             v-if="

package/src/shared/content-settings/components/slider-field/SliderField.mixin.ts

@@ -45,15 +45,6 @@ export default defineComponent({
       sliderValue: value,
     };
   },
-  watch: {
-    // The value can change outside the slider (e.g. a layout switch
-    // re-applying default padding, or undo) — keep the display in sync.
-    value(newValue: string | undefined) {
-      this.sliderValue = this.valueConvertor
-        ? this.valueConvertor.convert(newValue as any)
-        : newValue;
-    },
-  },
   computed: {
     innerMinLength(): number {
       return this.minLength ?? 0;

package/src/utils/minify-brand-page-metadata.ts

@@ -205,17 +205,7 @@ const minifyContent = (
   editor: Editor,
   assets?: AssetEntry[],
 ): MinifiedContent => {
-  const base: MinifiedContent = {
-    id: content.id,
-    type: content.type,
-    // Padding Control (BP-5687) — shared across every block, persist when set.
-    ...(typeof content.topPadding === 'number' && {
-      topPadding: content.topPadding,
-    }),
-    ...(typeof content.bottomPadding === 'number' && {
-      bottomPadding: content.bottomPadding,
-    }),
-  };
+  const base: MinifiedContent = { id: content.id, type: content.type };
   const c = content as any;
 
   // Helper: include imageUrl and its asset key if available

package/lib/src/brand-page/utils/content-padding.util.d.ts

@@ -1,13 +0,0 @@
-/**
- * Block top/bottom padding (BP-5687) as a bindable CSS style object. Defaults
- * live in the data — `content-settings` for new content, a migration for
- * existing — so this just reads the stored value (`?? 0` covers content from
- * before the field existed).
- */
-export declare const contentPaddingStyle: (content: {
-    topPadding?: number;
-    bottomPadding?: number;
-}) => {
-    paddingTop: string;
-    paddingBottom: string;
-};

package/lib/src/brand-page/utils/content-padding.util.js

@@ -1,10 +0,0 @@
-/**
- * Block top/bottom padding (BP-5687) as a bindable CSS style object. Defaults
- * live in the data — `content-settings` for new content, a migration for
- * existing — so this just reads the stored value (`?? 0` covers content from
- * before the field existed).
- */
-export const contentPaddingStyle = (content) => ({
-    paddingTop: `${content.topPadding ?? 0}px`,
-    paddingBottom: `${content.bottomPadding ?? 0}px`,
-});

package/lib/src/brand-page/utils/migration/migration-17.d.ts

@@ -1,2 +0,0 @@
-import type { MigrationData } from '../migration.util';
-export declare const migration17: (migrationResult: MigrationData) => Promise<MigrationData>;

package/lib/src/brand-page/utils/migration/migration-17.js

@@ -1,60 +0,0 @@
-import { BRAND_PAGE_TYPES } from '../../models';
-import { CONTACT_FORM_LAYOUT_DEFAULT_PADDING, TESTIMONIAL_DEFAULT_PADDING, bannerDisplayStyleDefaultPadding, } from '../../constants';
-/**
- * Padding Control backfill (BP-5699 Tier 2, BP-5700 Store): blocks that already
- * had hard-coded top/bottom spacing. Backfill that spacing as the block's explicit
- * topPadding/bottomPadding so existing pages render unchanged once the spacing
- * moves to the Padding Control sliders. Type-scoped and idempotent — only the
- * listed types, and only when the value isn't already set.
- *
- * Exception — `testimonial-section` (BP-5703): testimonials had 32px mobile / 80px
- * desktop and are intentionally collapsing to a single value across both breakpoints,
- * so the backfill sets that single value rather than preserving the old desktop 80px.
- * Existing testimonials will look tighter on desktop / roomier on mobile — by design.
- */
-const BLOCK_DEFAULT_PADDING = {
-    'images-section': 32,
-    'social-links': 32,
-    'shapes-section': 32,
-    'booking-section': 32,
-    'shop-section': 16,
-    'about-section': 32,
-    'testimonial-section': TESTIMONIAL_DEFAULT_PADDING,
-};
-export const migration17 = async (migrationResult) => {
-    const { brandPageModel } = migrationResult;
-    brandPageModel.pages.forEach((page) => {
-        page.contents.forEach((content) => {
-            // Contact Form and Banner previously varied their hard-coded spacing by
-            // layout/display style (full-bleed image layouts had no outer padding), so they're
-            // keyed by that rather than a single per-type value.
-            let value;
-            if (content.type === 'contact-form') {
-                value =
-                    CONTACT_FORM_LAYOUT_DEFAULT_PADDING[content.layoutStyle ?? ''];
-            }
-            else if (content.type === 'banner-section') {
-                value = bannerDisplayStyleDefaultPadding(content.displayStyle);
-            }
-            else if (content.type === 'text') {
-                // Text outer padding is brand-page-type specific: 12 on Brand Site, 0 on
-                // Brand Page / Brand Contact - backfill per type so existing pages render unchanged.
-                value =
-                    brandPageModel.brandPageType === BRAND_PAGE_TYPES.brandSite ? 12 : 0;
-            }
-            else {
-                value = BLOCK_DEFAULT_PADDING[content.type];
-            }
-            if (value === undefined) {
-                return;
-            }
-            if (content.topPadding === undefined) {
-                content.topPadding = value;
-            }
-            if (content.bottomPadding === undefined) {
-                content.bottomPadding = value;
-            }
-        });
-    });
-    return migrationResult;
-};

package/src/brand-page/utils/content-padding.util.test.ts

@@ -1,29 +0,0 @@
-import { describe, test, expect } from 'vitest';
-import { contentPaddingStyle } from './content-padding.util';
-
-describe('content-padding.util', () => {
-  test('contentPaddingStyle should map stored padding to px values', () => {
-    expect(contentPaddingStyle({ topPadding: 53, bottomPadding: 24 })).toEqual({
-      paddingTop: '53px',
-      paddingBottom: '24px',
-    });
-  });
-
-  test('contentPaddingStyle should default missing values to 0 for content created before the field existed', () => {
-    expect(contentPaddingStyle({})).toEqual({
-      paddingTop: '0px',
-      paddingBottom: '0px',
-    });
-    expect(contentPaddingStyle({ topPadding: 16 })).toEqual({
-      paddingTop: '16px',
-      paddingBottom: '0px',
-    });
-  });
-
-  test('contentPaddingStyle should keep an explicit 0 rather than treating it as unset', () => {
-    expect(contentPaddingStyle({ topPadding: 0, bottomPadding: 0 })).toEqual({
-      paddingTop: '0px',
-      paddingBottom: '0px',
-    });
-  });
-});

package/src/brand-page/utils/content-padding.util.ts

@@ -1,13 +0,0 @@
-/**
- * Block top/bottom padding (BP-5687) as a bindable CSS style object. Defaults
- * live in the data — `content-settings` for new content, a migration for
- * existing — so this just reads the stored value (`?? 0` covers content from
- * before the field existed).
- */
-export const contentPaddingStyle = (content: {
-  topPadding?: number;
-  bottomPadding?: number;
-}): { paddingTop: string; paddingBottom: string } => ({
-  paddingTop: `${content.topPadding ?? 0}px`,
-  paddingBottom: `${content.bottomPadding ?? 0}px`,
-});

package/src/brand-page/utils/migration/migration-17.ts

@@ -1,77 +0,0 @@
-import type { MigrationData } from '../migration.util';
-import type { BannerSectionContent, ContactFormContent } from '../../models';
-import { BRAND_PAGE_TYPES } from '../../models';
-import {
-  CONTACT_FORM_LAYOUT_DEFAULT_PADDING,
-  TESTIMONIAL_DEFAULT_PADDING,
-  bannerDisplayStyleDefaultPadding,
-} from '../../constants';
-
-/**
- * Padding Control backfill (BP-5699 Tier 2, BP-5700 Store): blocks that already
- * had hard-coded top/bottom spacing. Backfill that spacing as the block's explicit
- * topPadding/bottomPadding so existing pages render unchanged once the spacing
- * moves to the Padding Control sliders. Type-scoped and idempotent — only the
- * listed types, and only when the value isn't already set.
- *
- * Exception — `testimonial-section` (BP-5703): testimonials had 32px mobile / 80px
- * desktop and are intentionally collapsing to a single value across both breakpoints,
- * so the backfill sets that single value rather than preserving the old desktop 80px.
- * Existing testimonials will look tighter on desktop / roomier on mobile — by design.
- */
-const BLOCK_DEFAULT_PADDING: Partial<Record<string, number>> = {
-  'images-section': 32,
-  'social-links': 32,
-  'shapes-section': 32,
-  'booking-section': 32,
-  'shop-section': 16,
-  'about-section': 32,
-  'testimonial-section': TESTIMONIAL_DEFAULT_PADDING,
-};
-
-export const migration17 = async (
-  migrationResult: MigrationData,
-): Promise<MigrationData> => {
-  const { brandPageModel } = migrationResult;
-
-  brandPageModel.pages.forEach((page) => {
-    page.contents.forEach((content) => {
-      // Contact Form and Banner previously varied their hard-coded spacing by
-      // layout/display style (full-bleed image layouts had no outer padding), so they're
-      // keyed by that rather than a single per-type value.
-      let value: number | undefined;
-
-      if (content.type === 'contact-form') {
-        value =
-          CONTACT_FORM_LAYOUT_DEFAULT_PADDING[
-            (content as ContactFormContent).layoutStyle ?? ''
-          ];
-      } else if (content.type === 'banner-section') {
-        value = bannerDisplayStyleDefaultPadding(
-          (content as BannerSectionContent).displayStyle,
-        );
-      } else if (content.type === 'text') {
-        // Text outer padding is brand-page-type specific: 12 on Brand Site, 0 on
-        // Brand Page / Brand Contact - backfill per type so existing pages render unchanged.
-        value =
-          brandPageModel.brandPageType === BRAND_PAGE_TYPES.brandSite ? 12 : 0;
-      } else {
-        value = BLOCK_DEFAULT_PADDING[content.type];
-      }
-
-      if (value === undefined) {
-        return;
-      }
-
-      if (content.topPadding === undefined) {
-        content.topPadding = value;
-      }
-
-      if (content.bottomPadding === undefined) {
-        content.bottomPadding = value;
-      }
-    });
-  });
-
-  return migrationResult;
-};

package/src/brand-page-maker/convertors/contact-layout-style.convertor.test.ts

@@ -1,50 +0,0 @@
-import { describe, test, expect } from 'vitest';
-import { contactLayoutStyleConvertor } from './contact-layout-style.convertor';
-import {
-  ALIGN_CENTERED,
-  ALIGN_CARD_CENTER,
-  ALIGN_CARD_CENTER_IMAGE,
-  ALIGN_LEFT_FULL_IMAGE,
-  TEXT_ONLY,
-} from '@brand-page/constants';
-
-const configWithSelected = (selected: string) => ({
-  options: contactLayoutStyleConvertor
-    .convert(selected)
-    .options.map((option) => ({
-      ...option,
-      isSelected: option.value === selected,
-    })),
-});
-
-describe('contact-layout-style.convertor', () => {
-  test.each([
-    [ALIGN_CARD_CENTER, 32, 'center'],
-    [ALIGN_CENTERED, 32, 'center'],
-    [ALIGN_CARD_CENTER_IMAGE, 0, 'center'],
-    [TEXT_ONLY, 32, 'left'],
-    [ALIGN_LEFT_FULL_IMAGE, 0, 'left'],
-  ])(
-    'convertBack should apply the default padding and text alignment for %s',
-    (layoutStyle, padding, textAlign) => {
-      const result = contactLayoutStyleConvertor.convertBack(
-        configWithSelected(layoutStyle),
-      );
-
-      expect(result).toStrictEqual({
-        layoutStyle,
-        topPadding: padding,
-        bottomPadding: padding,
-        titleStyle: {
-          textAlign,
-        },
-        descriptionStyle: {
-          textAlign,
-        },
-        buttonTextStyle: {
-          textAlign: 'center',
-        },
-      });
-    },
-  );
-});