-
Notifications
You must be signed in to change notification settings - Fork 148
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
Fixed Background Image Visibility Issue on Profile Card (#31) #375
Fixed Background Image Visibility Issue on Profile Card (#31) #375
Conversation
✅ Deploy Preview for openfrontendmentor ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Thank you @syedfardeenjeelani for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@syedfardeenjeelani, Now background images are not working on mobile devices, so make some changes please then I will merge your branch.
sure |
@beRajeevKumar kindly review my pr |
@beRajeevKumar please review my pr |
What does this PR do?
This PR fixes the issue where the background images (
bg-top
andbg-bottom
) on the profile card disappear or become less visible on smaller screens. The issue is addressed by resizing the background images usingvw
units and adjusting their position to ensure visibility across different viewport sizes.Fixes #31
Type of change
How should this be tested?
bg-top
andbg-bottom
images on a mobile screen (max-width: 375px).Mandatory Tasks