Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made the blog images and description uniform in height #1435

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

r120dhiman
Copy link

Bugfix

#1434:

Screenshots/videos:
Screenshot of localhost

Screen.Recording.2025-02-28.at.02.13.59.mov

Summary

In this PR as per the requirement of the issue #1434 the size of all the images in the Blog page is made uniform.
I have added a max height if 200px on a div and wrapped the image inside the div.
To accommodate the change I have to make the overall blog card a little bigger that is 498px -> 510px.
added a height of 80px to title of the blog by which we have achieved the uniformness in the description of the blogs.

Does this PR introduce a breaking change?
The PR increases the overall UI much better and due to which the user experience becomes better.

@r120dhiman r120dhiman requested a review from a team as a code owner February 27, 2025 20:58
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to the JSON Schema Community. Thanks a lot for creating your first pull request!! 🎉🎉 We are so excited you are here! We hope this is only the first of many! For more details check out README.md file.

@idanidan29
Copy link

idanidan29 commented Feb 27, 2025

@r120dhiman Please read the comment I left on the issue. The issue hasn't been approved yet and hasn't been assigned to you.

@r120dhiman
Copy link
Author

Ok sir

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.

🐛 Bug: Standardize Blog Image Sizes
2 participants