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

Expanding the user list on smaller screens breaks the UI's vertical scrolling #136

Open
jeremyckahn opened this issue Aug 10, 2023 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@jeremyckahn
Copy link
Owner

jeremyckahn commented Aug 10, 2023

Originally posted by @africa552036012020255252355634346346346 in #135 (comment)

@jeremyckahn jeremyckahn added bug Something isn't working help wanted Extra attention is needed labels Aug 10, 2023
@africa552036012020255252355634346346346
Copy link

Adding a bit to this one: (in brave & cromite @ android, not firefox) the vertical scrolling is already somewhat broken when in a conversation even without expanding the user list or menu; neither the menu bar or the message field is at the actual top / bottom of the page, instead there is a gap creating a scrollbar.

(Unrelated but can be fix'd in the same go: especially on phone unless you have big hands or a small phone it's one hell of a stretch to reach all the way up to the menu bar so operating the app with just one hand comes with a slight amplified risk of dropping the phone.

Apps like Discord (on android at least) have solved this thru having the entire screen be draggable left n right, it only works in their dedicated app and not their web version tho.

A neat way of solving that apart from ^ would be to have a setting where the menu can be moved down to float at the bottom instead (above the message field).)

@jeremyckahn
Copy link
Owner Author

From @africa552036012020255252355634346346346 via #162:

Peerlist (& the entire page) never extending beyond the initial viewport (fixing #136) (& tiny scroll in there Discord style for when having many connections if it's not there already).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants