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

Replace all overflow-y scrolls #399

Closed
Masterjun3 opened this issue Sep 7, 2021 · 1 comment · Fixed by #614
Closed

Replace all overflow-y scrolls #399

Masterjun3 opened this issue Sep 7, 2021 · 1 comment · Fixed by #614
Milestone

Comments

@Masterjun3
Copy link
Collaborator

This is a debatable design feature. Currently, there are several elements with a maximum height, which show content in a scrollable block if the height exceeds the maximum value.

I personally agree with the feedback on the forums that this should be replaced with a different design. The block-scrolling can interrupt page-scrolling, and this is especially noticable on mobile devices where actual page-scrolling is only possible on the side edges of the screen.

For small limited blocks such as the featured movie or the news on the front page it should be asked if it's even necessary to limit the height.
For potentially larger blocks such as the submission text, a friendlier solution would be a label at the bottom saying "Read more", which when pressing displays more of the content.

@nattthebear
Copy link
Collaborator

I'm tentatively in agreement, but it'd be nice to see some design proposals.

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

Successfully merging a pull request may close this issue.

3 participants