Skip to content

Using drawer navigation leads to iOS browser double cursor #313

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

Closed
deyanskiq opened this issue Nov 23, 2021 · 7 comments
Closed

Using drawer navigation leads to iOS browser double cursor #313

deyanskiq opened this issue Nov 23, 2021 · 7 comments

Comments

@deyanskiq
Copy link

deyanskiq commented Nov 23, 2021

Using drawer and drawer-mobile daisyui classes leads to double cursor + iOS browser menu not hiding on scroll (which is the default behaviour).

I have noticed that on both my project and daisy ui website (screenshot attached).

Any suggestions how to fix this?

IMG_9347

@saadeghi
Copy link
Owner

@deyanskiq This doesn't seem to be a problem on the new iOS safari but I will look into that to see if there's a workaround...

@deyanskiq
Copy link
Author

Just tested on the latest iOS version (15.1), iPhone XR, and it seems that the behaviour is the same. Please have a look if there's a workaround.
IMG_DAE49AF820BE-1

@instanceofmel
Copy link

I can confirm that the problem still persists on my iPhone XR.

@saadeghi saadeghi added the iOS label Jan 17, 2022
saadeghi added a commit that referenced this issue Jan 17, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@sshquack
Copy link

Not sure if this is related but this might be related to the trackpad swipe gesture not working in drawer-content so I created #432

@deyanskiq
Copy link
Author

Hey @saadeghi, any update on the topic?

@saadeghi
Copy link
Owner

The problem is generally the 100vh height and it's not just a problem from daisyUI. Mobile browsers have dynamic sections on top or bottom that will show and hide based on the scroll position and that section is not calculated when we want a full height section and we use 100vh for it. I tried a few solutions but none of them were practical to use on production. There are new height units coming to CSS that will solve this but the browser support is not good yet.

@deyanskiq
Copy link
Author

deyanskiq commented Feb 23, 2022

Ok, thanks! Can you share a workaround that we can apply until then?

inorganik pushed a commit to inorganik/daisyui that referenced this issue Feb 7, 2023
inorganik pushed a commit to inorganik/daisyui that referenced this issue Feb 7, 2023
imgbot bot pushed a commit to Avensen/daisyui that referenced this issue Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants