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
Element Click and Element Clear need to handle user prompts (#1191) #1303
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch.
@whimboo: there's a merge conflict, but once that's fixed, this is good to go. |
It also requires tests. |
Yes, as mentioned in the initial comment tests will be added with https://bugzilla.mozilla.org/show_bug.cgi?id=1473814 soon. I will rebase to solve the conflict soon. |
As it looks like I was on an old revision which haven't had the webdriver-spec.html forwarding to index.html. I updated my PR to make it apply correctly. |
My patch for the tests has been landed on mozilla-autoland. Can someone please review and merge this PR? Thanks |
The open wpt-sync PR: web-platform-tests/wpt#13232 |
@whimboo this has merge conflicts, please rebase |
@AutomatedTester As the box above tells me there are no conflicts with the base branch. So what are you referring to? |
21d0de8
to
678bfb6
Compare
Fixes #1191. @shs96c mind reviewing this change?
Tests will be written via https://bugzilla.mozilla.org/show_bug.cgi?id=1473814.
Thanks.
Preview | Diff