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/0.7.0-rc1] Long handle in reports form pushes report status #1456

Closed
turtlegarden opened this issue Feb 7, 2023 · 3 comments · Fixed by #1464
Closed

[bug/0.7.0-rc1] Long handle in reports form pushes report status #1456

turtlegarden opened this issue Feb 7, 2023 · 3 comments · Fixed by #1464
Labels
bug Something isn't working frontend Frontend-related stuff

Comments

@turtlegarden
Copy link
Contributor

Describe the bug with a clear and concise description of what the bug is.

In the Reports user interface, a long user handle can push the report status to the side on smaller screen widths as shown below (with the expected behavior beside, for side-to-side comparison):

The above described graphical glitch, with a long user handle pushing the report status off the page

What's your GoToSocial Version?

v0.7.0-rc1

GoToSocial Arch

No response

Browser version

Bromite 108, WebKitGTK 2.38.3, Firefox 109

What happened?

The user handle does not wrap and pushes the report status out of the container.

What you expected to happen?

The user handle should wrap on small screens and not push the report status out of the container.

How to reproduce it?

Report a user from a long domain, or change the HTML to make a fake user from a long domain, and then load it on a small screen.

Anything else we need to know?

I already have a pull request mostly complete that fixes this issue, and tidies up some CSS bugs with wrapping user handles.

@turtlegarden turtlegarden added the bug Something isn't working label Feb 7, 2023
@turtlegarden turtlegarden changed the title [bug/0.7.0-rc1] Long handle in reports form pushes [bug/0.7.0-rc1] Long handle in reports form pushes report status Feb 8, 2023
@tsmethurst tsmethurst added the frontend Frontend-related stuff label Feb 8, 2023
@turtlegarden
Copy link
Contributor Author

about the PR— it fixes the issues but it uses some things like 60% max width when the sidebar wraps to the top, so it's somewhat hacky

@f0x52
Copy link
Contributor

f0x52 commented Feb 8, 2023

ah hmm, well you can open it as draft and I'll take a look (have some different ideas too)

@f0x52
Copy link
Contributor

f0x52 commented Feb 8, 2023

image
that should do it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend Frontend-related stuff
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants