Skip to content

Sumeru / #997 - Fix IterableEmbeddedView to match campaign preview#1034

Closed
sumeruchat wants to merge 1 commit intomasterfrom
sumeru/997-fix-embedded-view-campaign-preview-mismatch
Closed

Sumeru / #997 - Fix IterableEmbeddedView to match campaign preview#1034
sumeruchat wants to merge 1 commit intomasterfrom
sumeru/997-fix-embedded-view-campaign-preview-mismatch

Conversation

@sumeruchat
Copy link
Copy Markdown
Contributor

@sumeruchat sumeruchat commented Apr 7, 2026

Summary

  • Fix card view image height: change from 0dp constrained to wrap_content with adjustViewBounds for proper image sizing
  • Change image scaleType from centerCrop to fitCenter to match campaign preview rendering
  • Fix card text container from fixed 120dp height to wrap_content for dynamic sizing
  • Fix notification_view_border.xml where stroke and solid colors were swapped (border showed as background color and vice versa)

Test plan

  • Verify card embedded view displays images at correct aspect ratio matching campaign preview
  • Verify notification embedded view shows correct border and background colors
  • Verify banner embedded view is unaffected
  • Test with various image sizes/aspect ratios
  • Compare rendered views against campaign preview in Iterable dashboard

🤖 Generated with Claude Code

Fix card view image height to use wrap_content with adjustViewBounds
for proper image sizing. Change scaleType to fitCenter. Fix
notification_view_border.xml swapped stroke/solid colors.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sumeruchat sumeruchat self-assigned this Apr 7, 2026
@sumeruchat sumeruchat changed the title Sumeru - #997 Fix IterableEmbeddedView to match campaign preview Sumeru / #997 - Fix IterableEmbeddedView to match campaign preview Apr 7, 2026
@franco-zalamena-iterable franco-zalamena-iterable deleted the sumeru/997-fix-embedded-view-campaign-preview-mismatch branch April 8, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants