Skip to content
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

feat: add ability to specify offsets for JSHandle.click #7573

Merged
merged 1 commit into from Sep 20, 2021
Merged

Conversation

OrKoN
Copy link
Collaborator

@OrKoN OrKoN commented Sep 16, 2021

Until now, the click would be always sent to the middle
point of the target element. With this change, one can define
offsets relative to the border box of the elements and click
different areas of an element.

@OrKoN
Copy link
Collaborator Author

OrKoN commented Sep 16, 2021

I am still looking into the doclint failures but the code is ready to be reviewed I think. If you can tell what's wrong with the docs, I'd really appreciate :)

Copy link
Contributor

@jschfflr jschfflr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

docs/api.md Outdated Show resolved Hide resolved
@OrKoN OrKoN force-pushed the feat-offsets branch 2 times, most recently from 2a051cf to 6bb9f31 Compare September 16, 2021 08:48
@jschfflr
Copy link
Contributor

Should we get this landed or wait for a review from @mathiasbynens?

Copy link
Member

@mathiasbynens mathiasbynens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM % doc clarification

docs/api.md Outdated Show resolved Hide resolved
Until now, the click would be always sent to the middle
point of the target element. With this change, one can define
offsets relative to the border box of the elements and click
different areas of an element.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants