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

[BUG] Won't accept time update from mobile #1063

Open
lewsut opened this issue Jan 26, 2024 · 5 comments
Open

[BUG] Won't accept time update from mobile #1063

lewsut opened this issue Jan 26, 2024 · 5 comments
Labels
need investigation need investigation

Comments

@lewsut
Copy link

lewsut commented Jan 26, 2024

Works on PC browser, however mobile (android) time interface fails to take / update values.

Not that the return to the original time occurs when the screen is touched anywhere else after setting the new time.

XRecorder_26012024_144555.mp4
@unocelli
Copy link
Member

Hi, Thanks for the report
in master branch you should find a fix for this issue. new the datetime input control have a button to confirm.
image

@lewsut
Copy link
Author

lewsut commented Jan 28, 2024

Checked out the new version. The tick sits under the time text so it looks very odd no matter what the alignment is on chrome mobile , and clicking it does nothing, the result is exactly the same as in the video above.

Setup:

image

Firefox looks better, but doesn't work as intended, time isn't updated.

XRecorder_28012024_171253.mp4

Chrome looks poor, number always over the tick, and doesn't work as intended, time isn't updated.

XRecorder_28012024_171159.mp4

Also doesn't work on non mobile browser:

props

Works as expected (still looks weird on mobile with the tick behind the text) with " Enable update" disabled, however it's nice to not need another box to show an output.
image

@unocelli
Copy link
Member

I suppose that is not the only problem with mobile browser compatibility.

@lewsut
Copy link
Author

lewsut commented Feb 1, 2024

As this shows (from above), it doesn't work with "Enabled update" in a regular NON mobile browser.

300281070-bbd7972c-69b0-4897-8d42-092ee674c63f

@unocelli
Copy link
Member

unocelli commented Feb 4, 2024

with enabled update should be fixed, Thanks!

@unocelli unocelli added the need investigation need investigation label Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need investigation need investigation
Projects
None yet
Development

No branches or pull requests

2 participants