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

Bug: text overflow on footer #139

Closed
2 tasks done
dev-phantom opened this issue Jul 23, 2023 · 7 comments · Fixed by #148
Closed
2 tasks done

Bug: text overflow on footer #139

dev-phantom opened this issue Jul 23, 2023 · 7 comments · Fixed by #148
Assignees
Labels
🐛 bug Something isn't working good first issue Good for newcomers

Comments

@dev-phantom
Copy link
Contributor

Describe the bug

There is a text overflow in the link "insight.opensauced.pizza" on mobile view footer section
Screenshot_20230723-215451

Steps to reproduce

  1. Go to opensauced.pizza
  2. Scroll down to footer
  3. See error

Browsers

Chrome

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@dev-phantom dev-phantom added 🐛 bug Something isn't working 👀 needs triage labels Jul 23, 2023
@takanome-dev
Copy link

Tried different mobile screen sizes (using devtools) but couldn't reproduce it. Which size did you see it?

@dev-phantom
Copy link
Contributor Author

Tried different mobile screen sizes (using devtools) but couldn't reproduce it. Which size did you see it?

Screenshot_20230724-215100
Here is my mobile screen size

@wasimapinjari
Copy link
Contributor

That's not how you reproduce the problem. His phone screen is smaller than the dev tools simulation @takanome-dev.

You can see the problem by docking the developer tools on the side and dragging it all the way to the opposite side to simulate the smaller phone screens.

You will see the overflowing text that's what he is talking about.

Screenshot (196)

@wasimapinjari
Copy link
Contributor

You can fix that like so:

Screenshot (201)

@jpmcb jpmcb added good first issue Good for newcomers and removed 👀 needs triage labels Jul 28, 2023
@dev-phantom
Copy link
Contributor Author

.take

@github-actions
Copy link
Contributor

Thanks for taking this on! If you have not already, join the conversation in our Discord

@bdougie
Copy link
Member

bdougie commented Jul 29, 2023

Removing padding in the left column would be more ideal than wrapping on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants