feat(screencast): style action decorations, keep cursor visible - #42758
Pavel Feldman (pavelfeldman) wants to merge 1 commit into
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
- `showActions({ style: { point, highlight, title } })` takes CSS declarations for each decoration
- point marker and target highlight are opt-in, rendered via dedicated screencast elements
- deprecate `fontSize` in favor of `style.title`
- cursor stays at the last action point, survives navigations and travels along a slightly curved path
- macOS-style cursor
- `--cursor` for `video-start`, `--point-style`, `--highlight-style`, `--title-style` for `video-show-actions`
e252a42 to
51fb242
Compare
Test results for "tests 1"2 failed 7 flaky51979 passed, 1247 skipped Merge workflow run. |
🟢 Both failures are a known CDN flake — the PR is clearHi, I'm the Playwright bot and I triaged the red CI here. The two failures are DetailsOverall: nothing in this PR reaches browser downloading. The diff is screencast action decorations — Pre-existing flake / infra
No re-run needed for correctness; a retry should go green. Triaged by the Playwright bot - agent run |
Test results for "MCP"1 failed 8602 passed, 1446 skipped Merge workflow run. |
🟢 All three failures are known flakes — the PR is clearHi, I'm the Playwright bot and I triaged the red CI here. The MCP failure is DetailsOverall: the diff is screencast action decorations — Pre-existing flake / infra
A retry should go green; nothing here needs a code change. Triaged by the Playwright bot - agent run |
Summary
showActions({ style: { point, highlight, title } })takes CSS declarations per decoration; point marker and target highlight are now opt-infontSizein favor ofstyle.title--cursorforvideo-start,--point-style/--highlight-style/--title-styleforvideo-show-actions