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

docs(click): mention that trial emits keydown/keyup if modifier is given #32730

Closed
wants to merge 1 commit into from

Conversation

mxschmitt
Copy link
Member

* since: v1.11
- `trial` <[boolean]>

When set, this method only performs the [actionability](../actionability.md) checks, emits a `keydown`/`keyup` event if a `modifier` is passed and skips the action. Defaults to `false`. Useful to wait until the element is ready for the action without performing it.
Copy link
Member

Choose a reason for hiding this comment

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

What about mouse move, do we want to mention it for completeness? If not I'd reword it to say explicitly what it does not do (mouse down/up) and then say something like 'Note that if a modifier is passed, the call will fire keydown/keyup events .'

Copy link
Contributor

Test results for "tests 1"

2 failed
❌ [playwright-test] › runner.spec.ts:118:5 › should ignore subprocess creation error because of SIGINT
❌ [installation tests] › playwright-electron-should-work.spec.ts:21:5 › electron should work

1 flaky ⚠️ [chromium-library] › library/trace-viewer.spec.ts:254:1 › should have network requests

35678 passed, 668 skipped
✔️✔️✔️

Merge workflow run.

@muhqu
Copy link
Contributor

muhqu commented Sep 20, 2024

@mxschmitt @yury-s here is an alternative documentation fix: #32734

It addresses all cases where trail and modifiers are present, not just click.

@mxschmitt
Copy link
Member Author

Superseded by #32734

@mxschmitt mxschmitt closed this Sep 23, 2024
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.

3 participants