fix(components/lookup): lookup and selections modals use correct footer button spacing in modern theme (#4229)

[blackbaud-sky-build-user]

🍒 Cherry picked from #4229 fix(components/lookup): lookup and selections modals use correct footer button spacing in modern theme

AB#3704526

Summary by CodeRabbit

• Style
  • Updated button spacing in lookup and selection modal footers to be theme-aware, replacing fixed margins with conditional spacing that responds to the active theme selection.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 695f7d6 and 41ece68.

📒 Files selected for processing (2)

• libs/components/lookup/src/lib/modules/lookup/lookup-show-more-modal.component.html
• libs/components/lookup/src/lib/modules/selection-modal/selection-modal.component.html

---

📝 Walkthrough

Walkthrough

Two modal component templates are updated to apply margin styling conditionally based on the active theme rather than statically. The sky-margin-inline-compact class now binds to a theme condition, affecting only presentation logic without altering component behavior or event handling.

Changes

Cohort / File(s)	Summary
Modal Theme-Based Styling libs/components/lookup/src/lib/modules/lookup/lookup-show-more-modal.component.html, libs/components/lookup/src/lib/modules/selection-modal/selection-modal.component.html	Primary button margins changed from static class to dynamic theme-based binding. The sky-margin-inline-compact class now conditionally applies when the default theme is active, removing fixed styling in favor of theme-driven presentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested reviewers

• Blackbaud-ErikaMcVey
• Blackbaud-SteveBrush

Poem

🐰 The modals shine with theme-aware grace,
No static margins cluttering the place,
Sky margins dance with conditions so keen,
Default theme's the prettiest seen! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: fixing footer button spacing in lookup and selection modals for the modern theme, which aligns with the conditional theme-based styling changes in both components.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch cherry-pick_6453f81286687035440e68106dfc9f2762c2dc7e_1771874054996

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[nx-cloud]

View your CI Pipeline Execution ↗ for commit 41ece68

Command	Status	Duration	Result
nx build playground --baseHref=https://blackbau...	✅ Succeeded	2m 6s	View ↗
nx build code-examples-playground --baseHref=ht...	✅ Succeeded	2m 11s	View ↗
nx build integration --baseHref=https://blackba...	✅ Succeeded	51s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-02-23 19:24:59 UTC

[blackbaud-sky-build-user]

Storybook preview

Component Storybooks:

• ag-grid-storybook
• data-manager-storybook
• filter-bar-storybook
• layout-storybook
• lists-storybook
• lookup-storybook
• phone-field-storybook

Apps:

• code-examples-playground
• integration
• playground