Skip to content

fix: honor click modifiers on locator and element clicks - #6435

Open
aamoghS wants to merge 1 commit into
grafana:masterfrom
aamoghS:fix-browser-click-modifiers
Open

aamoghS wants to merge 1 commit into
grafana:masterfrom
aamoghS:fix-browser-click-modifiers

Conversation

@aamoghS

@aamoghS aamoghS commented Sep 19, 2026

Copy link
Copy Markdown

Summary

  • Click modifiers were parsed then dropped before the mouse click, so ControlOrMeta did not open a new tab.
  • Copy modifiers through ToMouseClickOptions and hold them on the keyboard for the click.

Closes #4312

Test plan

  • go test ./internal/js/modules/k6/browser/common/ -count=1 -run 'TestToMouseClickOptionsCopiesModifiers|TestKeyboardHoldModifiers'

Copy modifiers through ToMouseClickOptions and hold them on the keyboard for the duration of the mouse click so ControlOrMeta opens a new tab.

Closes grafana#4312
@aamoghS
aamoghS requested a review from a team as a code owner September 19, 2026 01:05
@aamoghS
aamoghS requested review from AgnesToulet and janHildebrandt98 and removed request for a team September 19, 2026 01:05
@github-actions

Copy link
Copy Markdown

Signed commits report

1 of 1 commit between master and fix-browser-click-modifiers could not be fully verified:

Commit Author Reason Message
846f5e48 aamoghS unsigned fix: honor click modifiers on locator and element clicks

This repository requires all commits to be signed. See GitHub docs on commit signature verification.

This branch has not been deployed

No deployments
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.

The modifiers option in click doesn't work

1 participant