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.

Sign up for GitHub

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

Jump to bottom

Website is unusable on mobile due to scroll container #1393

Open
jameshfisher opened this issue Nov 17, 2022 · 1 comment · May be fixed by #1535
Open

Website is unusable on mobile due to scroll container #1393

jameshfisher opened this issue Nov 17, 2022 · 1 comment · May be fixed by #1535

Comments

@jameshfisher
Copy link

Using Chrome on Android, scroll is unusable. The page seems to be wrapped in a container that has its own scroll bar.
@SaberStrat
Copy link

Yes, scrolling gets sticky on iOS too. It’s super annoying to get the site scrollable again.

It‘s been like this ever since I started reading the site 2021.

@antdking antdking mentioned this issue Jun 20, 2023
Open
lightster added a commit to lightster/helm-www that referenced this issue Dec 9, 2023
@lightster
Fix web site scrolling on mobile browsers
7bd3411 
Scrolling pages on the Helm web site is difficult on mobile devices due
to nested scrolling containers. This changeset removes the nested
scrolling containers.

The fix to remove the nested scrolling containers caused pages to start
scrolling horizontally because the content of the pages have negative
margins, which makes the page wider than 100%. The rest of this
changeset addresses the negative margins by removing them and/or
offsetting them in each element that was causing the page to horizontal
scroll.

Fixes helm#741
Fixes helm#1393
Fixes helm#1464

Signed-off-by: Matt Light <[email protected]>
@lightster lightster linked a pull request Dec 9, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix web site scrolling on mobile browsers lightster/helm-www
2 participants
@jameshfisher @SaberStrat