Skip to content

Conversation

@Draiken
Copy link

@Draiken Draiken commented Jan 5, 2026

Allows us to use time strings like 10:00 as values for a time input.

I've spent a lot of time thinking I had a bug because a simple fill_in "time_field", with: "10:00" didn't work. It seems to expect a time object only but that's not mentioned anywhere AFAIK.

This adds the ability to set times as strings which would work normally in JS element.value = "10:00"

@albiere
Copy link

albiere commented Jan 6, 2026

Just yesterday I moved from selenium to cuprite and I got some failed tests because of this. Had no idea a Time object was needed for the fill_in method. 👍 for this PR.

@Draiken
Copy link
Author

Draiken commented Jan 6, 2026

@albiere after creating the PR and having previously searched for this in the issues, I noticed there are other PRs doing similar changes. All from long ago, with little/no feedback. The GH actions fail with an unrelated issue.

I'm afraid this project is not active anymore or maybe the maintainers are on vacation.

All this to say that we shouldn't expect this to be merged any time soon.

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