Skip to content

Conversation

@jeetburman
Copy link

@jeetburman jeetburman commented Dec 28, 2025

Description

  • Added 2rem padding to the image in CTA_FullWidth component
  • Resolves issue with cramped Layer5 logo appearance in blog CTAs
  • Improves visual spacing and overall design quality

Notes for Reviewers
I wasnt really able to test the changes locally.
Pls let me know if padding needs to be more increased or decreased.

Signed commits

  • Yes, I signed my commits.

- Added 2rem padding to the image in CTA_FullWidth component
- Resolves issue with cramped Layer5 logo appearance in blog CTAs
- Improves visual spacing and overall design quality

Signed-off-by: Your Name <your.email@example.com>
Signed-off-by: Jeet Burman <jeetburrman@gmail.com>
@jeetburman jeetburman force-pushed the fix/cta-fullwidth-image-padding branch from 80e1891 to dde79a8 Compare December 28, 2025 16:12
@l5io
Copy link
Contributor

l5io commented Dec 28, 2025

🚀 Preview for commit dde79a8 at: https://69515a4757fa504da956ab31--layer5.netlify.app

@Kevindua26
Copy link

Kevindua26 commented Dec 28, 2025

Screenshot_20251228_223908_Chrome.jpg

Seems there're still some changes needed!
Look at the text part of the component.

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

Please ensure the mobile view is also handled properly.

@Kevindua26
Copy link

Kevindua26 commented Dec 28, 2025

Screenshot_20251228_223908_Chrome.jpg

Seems there're still some changes needed!
Look at the text part of the component.

This one is the tablet view.

@yugalkaushik
Copy link

Screenshot_2025-12-28-23-05-29-95_e4424258c8b8649f6e67d283a50a2cbc
For mobile view as well changes are needed

@jeetburman
Copy link
Author

I will look into this.

Signed-off-by: Jeet Burman <jeetburrman@gmail.com>
@jeetburman jeetburman force-pushed the fix/cta-fullwidth-image-padding branch from cda2932 to fc13081 Compare December 28, 2025 18:46
@l5io
Copy link
Contributor

l5io commented Dec 28, 2025

🚀 Preview for commit fc13081 at: https://69517e827a516960166a8668--layer5.netlify.app

@jeetburman
Copy link
Author

jeetburman commented Dec 28, 2025

The texts are appearing correctly. Hence, the bug is resolved. However, in phone view... the logo seems to overflow.
Screenshot 2025-12-29 003431

I think this is a problem...

Signed-off-by: Jeet Burman <jeetburrman@gmail.com>
@jeetburman jeetburman force-pushed the fix/cta-fullwidth-image-padding branch from c59ae43 to b9f117a Compare December 28, 2025 20:08
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.

5 participants