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

Fix scrolling in mobile browser #50

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

akramghaleb
Copy link

Hi,

There was problem in scrolling in mobile app. I have fix it with setTimeout in onStart Event.

Thanks

@akramghaleb
Copy link
Author

Hi,

There was problem in scrolling in mobile app. I have fix it with setTimeout in onStart Event.

Thanks

@mokhosh
Copy link
Owner

mokhosh commented Jul 13, 2024

Hey,

Please explain what the issue was and attach a video if necessary.

Also there are so many unrelated changes in this commit.

And the pr is from your main branch.

@akramghaleb
Copy link
Author

Title: Fix Mobile Scrolling Issue

Description:
This Pull Request addresses the issue where scrolling on mobile devices was not functioning correctly. The fix includes changes to the touch event handling and CSS adjustments to ensure smooth scrolling.

Changes Made:

  • Adjusted touch event handling for better responsiveness.
  • Updated CSS for improved scrolling behavior.

Notes:
Please review the changes in the context of the provided video to confirm that the issue is resolved. This PR is based on the fix-scroll-issue branch.

@akramghaleb
Copy link
Author

Screen_Recording_20240714_090704_Chrome.mp4

@akramghaleb
Copy link
Author

After fixing the issue. scrolling work fine

Screen_Recording_20240714_090814_Chrome.mp4

@akramghaleb
Copy link
Author

See the videos before and after that demonstrates the issue and how the fix resolves it.

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 this pull request may close these issues.

2 participants