Skip to content

Conversation

vikas-kumar-git
Copy link
Contributor

the image was missing a max-height setting as max-width setting was given due to which the problem of image getting out of its parent container occurred when there was not enough device height which caused scrollbar to appear

Fixes Issue

My PR closes #issue_number_here
747

👨‍💻 Changes proposed(What did you do ?)

added tailwind css in img tag

✔️ Check List (Check all the applicable boxes)

  • [] My code follows the code style of this project.
  • [] This PR does not contain plagiarized content.
  • [] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

the image was missing a max-height setting as max-width setting was given due to which the problem of image getting out of its parent container  occurred when there was not enough device  height which caused scrollbar to appear
Copy link

vercel bot commented Mar 4, 2025

@vikas-kumar-git is attempting to deploy a commit to the dunsin's projects Team on Vercel.

A member of the Team first needs to authorize it.

</div>
<div className="h-full bg-secondary w-2/4 sm:flex hidden items-center justify-center px-4">
<img src="/landing page image.jpg" className="h-auto rounded-full w-auto object-cover" />
<img src="/landing page image.jpg" className="h-auto max-h-screen rounded-full w-auto object-cover" />
Copy link
Contributor

Choose a reason for hiding this comment

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

can you attach the screenshots of before <> after

@vikas-kumar-git vikas-kumar-git marked this pull request as ready for review March 8, 2025 07:50
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.

2 participants