Skip to content

feat(image): split logic into "fill" and "default" variants with fallback size resolution#153

Merged
eugeneshulga869 merged 2 commits into
developfrom
new-image
Jun 23, 2025
Merged

feat(image): split logic into "fill" and "default" variants with fallback size resolution#153
eugeneshulga869 merged 2 commits into
developfrom
new-image

Conversation

@eugeneshulga869
Copy link
Copy Markdown
Collaborator

This PR refactors the Image component logic to support two distinct variants:

Changes:
• Split image rendering logic into:
• "fill" variant — image takes size from the parent container.
• "default" variant — requires explicit image dimensions.
• Primary image dimensions now come from Uniform assets (resolvedImage.width/height).
• Fallback logic added using probe-image-size to retrieve image size by URL when no size metadata is available.
• Focal point support integrated (used in image transformation).

Implementation Notes:
• probe-image-size is used only when dimensions are missing from the asset.
• No impact on public API – backward compatible.

@eugeneshulga869 eugeneshulga869 self-assigned this Jun 20, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
csk-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2025 1:36pm
csk-marketing-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2025 1:36pm
csk-marketing-site-dev ✅ Ready (Inspect) Visit Preview Jun 20, 2025 1:36pm
csk-template-coffee-shop-ai-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2025 1:36pm
csk-template-coffee-shop-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2025 1:36pm
csk-v-next-dxa-template ❌ Failed (Inspect) Jun 20, 2025 1:36pm
csk-v-next-radiant-template ❌ Failed (Inspect) Jun 20, 2025 1:36pm
csk6-jd-preview ❌ Failed (Inspect) Jun 20, 2025 1:36pm
demos-csk-storybook-prod ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2025 1:36pm
dxa-demo ❌ Failed (Inspect) Jun 20, 2025 1:36pm
eugene-space-csk ❌ Failed (Inspect) Jun 20, 2025 1:36pm
fsdemo ❌ Failed (Inspect) Jun 20, 2025 1:36pm
java-drip-april-demo ❌ Failed (Inspect) Jun 20, 2025 1:36pm
qorvo-demo ❌ Failed (Inspect) Jun 20, 2025 1:36pm
website-digital-experience-assembly ❌ Failed (Inspect) Jun 20, 2025 1:36pm

@eugeneshulga869 eugeneshulga869 merged commit 213e8b8 into develop Jun 23, 2025
10 of 19 checks passed
@eugeneshulga869 eugeneshulga869 deleted the new-image branch June 23, 2025 06:23
eugeneshulga869 added a commit that referenced this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants