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

Fix the images on the blog cards #1591

Closed
YoshithaRathnayake opened this issue Dec 28, 2023 · 6 comments · Fixed by #1623
Closed

Fix the images on the blog cards #1591

YoshithaRathnayake opened this issue Dec 28, 2023 · 6 comments · Fixed by #1623
Assignees

Comments

@YoshithaRathnayake
Copy link
Member

Describe the bug
Currently, the images appear stretched and it would be better to have them properly cropped.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://sefglobal.org/engineering-team/'
  2. Scroll down to 'Blog Articles from Our Engineers'
  3. See error

Expected behavior
Fix the images on the blog cards using bootstrap

Screenshots
image

@YoshithaRathnayake
Copy link
Member Author

@jayasanka-sack this is the issue u wanted to create right?

@jayasanka-sack
Copy link
Member

@YoshithaRathnayake Mmm… I cannot recall.

@jayasanka-sack
Copy link
Member

Anyway, this is a good catch! This needs to be fixed

@mkskaushalya
Copy link
Contributor

mkskaushalya commented Apr 22, 2024

Hello @YoshithaRathnayake, I can fix this issue.

When inserting an image here, instead of inserting it directly, please insert it into a div with a fixed size and give the correct size to that div. This issue can be solved.

image

@YoshithaRathnayake
Copy link
Member Author

Hello @YoshithaRathnayake, I can fix this issue.

When inserting an image here, instead of inserting it directly, please insert it into a div with a fixed size and give the correct size to that div. This issue can be solved.

image

If I assign you can you fix it? @mkskaushalya

@ifhammohamed
Copy link
Contributor

Hi @anjula-sack @YoshithaRathnayake !

This is the issue which is I solved & here's the PR of it #1623

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 a pull request may close this issue.

4 participants