Skip to content

Update the way to click an element #121

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update the way to click an element #121

wants to merge 1 commit into from

Conversation

@omarlopesino
Copy link

@omarlopesino omarlopesino commented Feb 15, 2023

Problem description

When the click method is executed on a checkbox in Selenium 4.8.0, the following error appears:

non-positive contentLength: 0

Steps to reproduce

Call Webdriver\Element::click method on a checkbox with this library, using Selenium 4.8.0.

Solution

Send the :id attribute in the POST request. This solution is based in how this is done at php-webdriver/webdriver
@robocoder
Copy link
Member

robocoder commented Feb 16, 2023

This PR actually breaks click. This library is not a fork of php-webdriver/webdriver, and specifically doesn't use the :id placeholder.
@robocoder robocoder closed this Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@omarlopesino @robocoder