Skip to content

Time input overflowing fix #468

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
wants to merge 2 commits into from
Closed

Conversation

karthikk0811
Copy link

Screenshot 2025-05-05 at 4 34 54 PM Fixed the overflowing time input as shown in the below image. Screenshot 2025-05-05 at 4 35 33 PM

@iM-GeeKy
Copy link
Collaborator

iM-GeeKy commented May 5, 2025

@karthikk0811 Thanks for the PR! Would you mind taking screenshots on iOS and Android just to make sure they look correct as well?

@karthikk0811
Copy link
Author

Screenshot 2025-05-06 at 10 33 53 AM
This is the screenshot from ios.
Screenshot 2025-05-06 at 10 28 40 AM
and this is from the andriod.

I made a minor adjustment to the layout. On iOS and Android, the left padding was unnecessary, so I’ve conditionally applied it based on the platform. The screenshots above show the updated appearance on both iOS and Android. The web layout remains unchanged, as previously shared.

@iM-GeeKy
Copy link
Collaborator

iM-GeeKy commented May 6, 2025

Screenshot 2025-05-06 at 5 17 59 AM

It looks like the time still has an offset, particularly the double "00". Would you mind fixing that? Looking good otherwise!

@iM-GeeKy
Copy link
Collaborator

iM-GeeKy commented May 6, 2025

Screenshot 2025-05-06 at 5 27 13 AM

Also, what browser are you using? I'm trying to reproduce your issue and i'm not seeing it on the master branch using Chrome 🤔

@karthikk0811
Copy link
Author

Apologies for the confusion earlier. I just realized that I was using an older version (v0.22.36), which caused the issue. Everything is working as expected now after updating. Thank you for your time and support!

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