Skip to content

style: fix image cropping in platform engineering benefits section - #7972

Open
AnkitRewar11 wants to merge 1 commit into
layer5io:masterfrom
AnkitRewar11:fix/srl-image-cropped-platform-engineering
Open

style: fix image cropping in platform engineering benefits section#7972
AnkitRewar11 wants to merge 1 commit into
layer5io:masterfrom
AnkitRewar11:fix/srl-image-cropped-platform-engineering

Conversation

@AnkitRewar11

@AnkitRewar11 AnkitRewar11 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Description
This PR fixes #7951

Fixed the SRL-image-0.svg (catalog image) cropping issue in the "Leverage Ready-to-Use Building Blocks" section on the Platform Engineering page. The .img-wrapper img CSS had object-fit: cover set across all breakpoints, which was clipping the image inside its fixed-size container. Changed it to object-fit: contain so the full SVG is now visible and properly fitted within its container.

Notes for Reviewers
Tested locally on /solutions/platform-engineering across desktop, tablet (max-width: 767px), and mobile (max-width: 467px) breakpoints image now displays fully without cropping at all sizes.

Signed commits

  • Yes, I signed my commits.

Before
Screenshot 2026-08-12 at 6 44 50 AM (1)

After
Screenshot (682)

Summary by CodeRabbit

  • Bug Fixes
    • Updated platform images to fit fully within their containers across desktop and mobile layouts.
    • Prevented image cropping at responsive breakpoints.

Signed-off-by: ankitrewar11 <rewarankit18@gmail.com>
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a400fc97-b0cf-4356-81c2-cf69b08ba7c0

📥 Commits

Reviewing files that changed from the base of the PR and between 2ccf887 and 1fe843c.

📒 Files selected for processing (1)
  • src/sections/Platform-Engineering/platform.js

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The platform image styles change from object-fit: cover to object-fit: contain at the base, 767px, and 467px breakpoints.

Changes

Platform image fitting

Layer / File(s) Summary
Responsive image styling
src/sections/Platform-Engineering/platform.js
The StyledRow image styles use contain at all responsive breakpoints. The full image remains visible within its container.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 1fe84

This localized styling change adjusts image fitting so the full catalog image is visible; no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The change uses object-fit: contain at all breakpoints, which makes the complete SRL-image-0.svg visible as required by issue #7951.
Out of Scope Changes check ✅ Passed The pull request only changes image fitting in the targeted component and contains no unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the fix for image cropping in the Platform Engineering benefits section.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://layer5.io/pr-preview/pr-7972/

@dhruveshmishra dhruveshmishra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minimal changes,LGTM

@emphor11 emphor11 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR directly fixes the reported cropping issue, the chosen CSS behavior is appropriate, the before/after evidence shows the catalog image is now fully visible.

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.

SRL Image is Cropped in the Ready-to-Use Building Blocks Section

3 participants