Skip to content

[UI] Fix blog card thumbnail alignment in list viewfix: improve blog card thumbnail alignment in list view#7745

Open
Saanvim11 wants to merge 4 commits into
layer5io:masterfrom
Saanvim11:fix-blog-thumbnail-alignment
Open

[UI] Fix blog card thumbnail alignment in list viewfix: improve blog card thumbnail alignment in list view#7745
Saanvim11 wants to merge 4 commits into
layer5io:masterfrom
Saanvim11:fix-blog-thumbnail-alignment

Conversation

@Saanvim11
Copy link
Copy Markdown
Contributor

Description

This PR fixes inconsistent blog card thumbnail alignment in list view.

Changes

  • Replaced object-fit: contain with object-fit: cover
  • Updated image sizing to ensure thumbnails fill their containers consistently
  • Removed uneven spacing caused by varying image aspect ratios

This addresses the layout inconsistency discussed in #7743.

@rishiraj38 rishiraj38 added the issue/dco Commit sign-off instructions label May 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 left a comment

Choose a reason for hiding this comment

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

@Saanvim11 The DCO check is failing. Also, we already have a prop specifically handling the list view, so why are we adding object-fit properties again here?

I believe these changes might end up affecting the grid view as well.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

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

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@Saanvim11 Saanvim11 closed this May 21, 2026
@Saanvim11 Saanvim11 reopened this May 21, 2026
@rishiraj38
Copy link
Copy Markdown
Member

@Saanvim11 Please make a signed commit using git commit -s -m.

Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 left a comment

Choose a reason for hiding this comment

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

Please apply the changes requested.

Comment thread src/components/Card/Card.style.js Outdated
Comment thread src/components/Card/Card.style.js
Comment thread src/components/Card/Card.style.js
Comment thread src/components/Card/Card.style.js Outdated
Comment thread src/components/Card/Card.style.js Outdated
Comment thread src/components/Card/Card.style.js Outdated
Comment thread src/components/Card/Card.style.js Outdated
Comment thread src/components/Card/Card.style.js Outdated
Co-authored-by: Rishi Raj <180109771+rishiraj38@users.noreply.github.com>
Signed-off-by: Saanvim11 <saanvimoolchandani01@gmail.com>
@rishiraj38
Copy link
Copy Markdown
Member

@saurabhraghuvanshii @Rajesh-Nagarajan-11 Please take a look. Apart from the DCO, everything else looks good to me.

@Rajesh-Nagarajan-11
Copy link
Copy Markdown
Member

@Saanvim11 Kindly sign off your commits please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

issue/dco Commit sign-off instructions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants