[Docs] Fix alignment of Be Recognized card background illustration - #1165
Conversation
|
Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack. |
There was a problem hiding this comment.
Code Review
This pull request updates SCSS styles for landing project logos and video carousels. Specifically, it isolates the styling of .recognition-program-logo to apply custom background sizes and a minimum height, and nests carousel-related classes under .related-videos while updating transition effects, colors, and active states for indicators. There are no review comments, and I have no additional feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Signed-off-by: piyushtripathi9424 <piyush.miraibanglore@gmail.com>
e881e01 to
f67e4ce
Compare
|
@piyushtripathi9424 LGTM🚀 |
Thankyou ❤️ |
|
@piyushtripathi9424 Thank you for your contribution! Let's discuss this during the website call today (20 July) at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
Thank you! I’ll add it to the meeting agenda/minutes and discuss it during today’s website call at 5:30 PM IST. 🙂 |
|
@piyushtripathi9424 any updates? |
|
@Maanvi212006 No updates from my side. The PR is ready please let me know if any changes are needed. Thanks! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe landing project now applies dedicated background positioning, sizing, repetition, and minimum-height rules to the recognition program logo. The logo is removed from the shared catalog and academy selector. ChangesRecognition Program Logo Styling
Estimated code review effort: 1 (Trivial) | ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
Description
This PR fixes the alignment and sizing issue with the background illustration in the "Be Recognized" card (#1156).
Root Cause: The
background-size: 50%property was being applied to both the black semi-opaque background and therecognition-program.pnglogo. Because the logo is much taller than it is wide, keeping its width at 50% caused its height to stretch and peek out from behind the square black background.Fix: Extracted
.recognition-program-logointo its own CSS rule and increased the size of the black semi-opaque background to75%while preserving the logo size at50%. This ensures the logo is fully contained within the background circle, visually matching the other resource cards.Fixes #1156
Screenshots 📸
Summary by CodeRabbit