diff --git a/src/templates/EmailEditorContent/EmailSubComponentSpacer.tsx b/src/templates/EmailEditorContent/EmailSubComponentSpacer.tsx index 8dd14677..d07c2f73 100644 --- a/src/templates/EmailEditorContent/EmailSubComponentSpacer.tsx +++ b/src/templates/EmailEditorContent/EmailSubComponentSpacer.tsx @@ -42,7 +42,7 @@ const sizeForSubComponentKind = ( case 'RulesRightsRegulations': return 'extraLarge' case 'Title': - return 'large' + return shouldShowNext ? 'large' : undefined case 'Intro': return 'medium' case 'SupplementalContent': diff --git a/src/templates/EmailEditorContent/__tests__/EmailSubComponentSpacer.test.tsx b/src/templates/EmailEditorContent/__tests__/EmailSubComponentSpacer.test.tsx index a2dcbd9f..2cd81382 100644 --- a/src/templates/EmailEditorContent/__tests__/EmailSubComponentSpacer.test.tsx +++ b/src/templates/EmailEditorContent/__tests__/EmailSubComponentSpacer.test.tsx @@ -54,12 +54,35 @@ describe('EmailSubComponentSpacer', () => { expect(size).toBeUndefined() }) - it('is large when the subcomponent is Title', () => { - const size = renderWithSubComponents({ - currentSubComponent: 'Title', - nextSubComponent: 'ProgramName', + describe('Title', () => { + it('is large when the next subcomponent should be shown', () => { + const size = renderWithSubComponents({ + currentSubComponent: 'Title', + nextSubComponent: 'ProgramName', + }) + expect(size).toEqual(`${spacingCellSizes.large}px`) + }) + + it('is nothing when the next subcomponent should not be shown', () => { + const nextSubComponent = buildUniqueEmailSubComponent('Body', { + kind: 'ProgramName', + }) + const { baseElement } = render( + + + , + { + wrapper: emailPartWrapper, + }, + ) + const spacer = baseElement.querySelector('td') + expect(spacer).toBeNull() }) - expect(size).toEqual(`${spacingCellSizes.large}px`) }) it('is nothing when the subcomponent is ProgramName', () => {