Skip to content

fix(types): expose signal option of Locator.elementHandle - #42520

Closed
Ashraf Ali (ashrafiucse) wants to merge 1 commit into
microsoft:mainfrom
ashrafiucse:fix-element-handle-signal-type
Closed

fix(types): expose signal option of Locator.elementHandle#42520
Ashraf Ali (ashrafiucse) wants to merge 1 commit into
microsoft:mainfrom
ashrafiucse:fix-element-handle-signal-type

Conversation

@ashrafiucse

Copy link
Copy Markdown
Contributor

Summary

  • Add the missing signal option to the hand-written Locator.elementHandle override so it shows up in the generated types
  • The option is documented (since v1.62) and honored by the client implementation (client/locator.ts passes TimeoutOptions into frame.waitForSelector), but the override literal omitted it and suppressed the doc-generated signature

The signal option is documented and honored by the client implementation
but was missing from the generated types because the hand-written
override did not list it.
@dgozman

Copy link
Copy Markdown
Collaborator

Ashraf Ali (@ashrafiucse) Please file an issue before sending a PR, according to the contributing guide.

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