-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add Python and Ruby example for Offset from Element (Center Origin) #1259
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
Conversation
✅ Deploy Preview for jovial-austin-42fe02 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Good catch! I never updated these. Can you also remove the section preceding it since we no longer support offset from upper-left? |
@titusfortner, I removed the preceding section. |
@titusfortner, I see you merged SeleniumHQ/selenium#11402, which fixes the function comments. However, the documentation is not updated correspondingly, see: https://www.selenium.dev/documentation/webdriver/actions_api/mouse/#move-by-offset I wonder why this PR is closed instead of being merged. Thank you. |
@titusfortner Just pinging this PR, as it's been a month since the last reply. I'm still wondering why this PR is closed instead of being merged, since the issue is still present. Thank you. |
Hmm, probably I just didn't look closely enough at it. Thanks for the ping. |
Just pinging this PR again, as it's been another 2 weeks since the last reply. |
Just pinging this PR, as it's been 1.5 months since the last reply. |
Just merged the |
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.
Thank you, @j3soon!
Related: SeleniumHQ/selenium#10261
Description
Add examples for
move_to_element_with_offset
for Selenium >= 4.3.Link: https://www.selenium.dev/documentation/webdriver/actions_api/mouse/#move-by-offset
Motivation and Context
Correspond to the fix: SeleniumHQ/selenium#11402
Types of changes
Checklist