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 : Images getting cropped when help center is active in small devices #1573

Conversation

aritradevelops
Copy link
Contributor

Fixes #1428

Issue

In small devices when opening the help center the top level div of the New Background Component was shrinking its size down and was not starting itself from top:0, therefore the images inside that was also starting from that level, which to user, seems like the image is getting cropped.

Images

Screenshot 2024-10-20 at 11 08 06 AM Screenshot 2024-10-20 at 11 12 19 AM

What does this PR do

changing its position from absolute to fixed sticks it to the top always, therefore the image will start from the top and look normal

Images

Screenshot 2024-10-20 at 11 10 54 AM Screenshot 2024-10-20 at 11 11 39 AM Screenshot 2024-10-20 at 11 14 12 AM

How to test this

  1. Visit https://app.dub.co/onboarding/welcome
  2. Check in Mobile or Inspect and Choose Mobile Screens
  3. Click the Help Center Icon ( ? ) at the bottom left

Oss

As you can see many people have already tried fixing it and it was not a easy to find issue so I would request you to award a bit more point for this fix, thanks in advance!

Copy link

vercel bot commented Oct 20, 2024

@aritradevelops is attempting to deploy a commit to the Dub Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Oct 20, 2024

CLA assistant check
All committers have signed the CLA.

@aritradevelops
Copy link
Contributor Author

@steven-tey please have a look, thanks!

@steven-tey steven-tey self-requested a review October 21, 2024 22:56
@steven-tey
Copy link
Collaborator

amazing fix and writeup!

@steven-tey steven-tey merged commit dc9e145 into dubinc:main Oct 21, 2024
4 of 5 checks passed
Copy link

oss-gg bot commented Oct 21, 2024

Awarding aritradevelops: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/aritradevelops

@steven-tey
Copy link
Collaborator

/award 300

Copy link

oss-gg bot commented Oct 21, 2024

Awarding aritradevelops: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/aritradevelops

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

Successfully merging this pull request may close these issues.

[🕹️] Background image gets cropped when Opening the Help Center in Mobile Responsive
3 participants