Skip to content

fix(design)!: scope callout body paragraph margins to direct children - #4617

Merged
xelaint merged 1 commit into
developfrom
bug/callout-body-paragraph
Jul 27, 2026
Merged

fix(design)!: scope callout body paragraph margins to direct children#4617
xelaint merged 1 commit into
developfrom
bug/callout-body-paragraph

Conversation

@xelaint

@xelaint xelaint commented Jul 27, 2026

Copy link
Copy Markdown
Member

PR Checklist

  • Commit message follows our contributing guidelines
  • Tests added/updated (for bug fixes/features)
  • Documentation added/updated (for bug fixes/features)

PR Type

  • Bug fix
  • Feature
  • Style update
  • Refactor
  • Test
  • Build
  • CI
  • Docs
  • Performance
  • Other (please describe)

Current behavior

Fixes: #4616

New behavior

Breaking change?

  • Yes
  • No

BREAKING CHANGE: DaffCalloutComponent no longer resets the margins of paragraphs nested inside child components within [daffCalloutBody]. Previously the first and last <p> under the body received margin-top: 0 and margin-bottom: 0 at any depth. The reset now applies only to paragraphs that are direct children of [daffCalloutBody]. If you relied on the old behavior to flatten spacing in nested content, apply those margins in your own styles.

Additional context

BREAKING CHANGE: `DaffCalloutComponent` no longer resets the margins of paragraphs nested inside child components within `[daffCalloutBody]`. Previously the first and last `<p>` under the body received `margin-top: 0` and `margin-bottom: 0` at any depth. The reset now applies only to paragraphs that are direct children of `[daffCalloutBody]`. If you relied on the old behavior to flatten spacing in nested content, apply those margins in your own styles.
@xelaint
xelaint requested a review from a team as a code owner July 27, 2026 20:33
@xelaint xelaint added package: design @daffodil/design 🚨 breaking changes This PR or issue will cause a breaking change labels Jul 27, 2026
@xelaint
xelaint merged commit fedce39 into develop Jul 27, 2026
16 checks passed
@gray-bot gray-bot mentioned this pull request Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 breaking changes This PR or issue will cause a breaking change package: design @daffodil/design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Callout body paragraph margin resets leak into nested paragraphs

1 participant