Skip to content

feat(video): expand annotate config with test status and action options#39813

Merged
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:status_line
Mar 24, 2026
Merged

feat(video): expand annotate config with test status and action options#39813
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:status_line

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • Restructure video annotate config from { delay?: number } to { action?: { delay?: number }, test?: { level?: 'file' | 'title' | 'step' } }
  • action controls element highlight annotations during input actions
  • test controls screencast status overlay showing test file, title, and/or step info
  • Status level is cumulative: file shows file name, title adds test title, step adds current step

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@microsoft microsoft deleted a comment from github-actions bot Mar 24, 2026
@microsoft microsoft deleted a comment from github-actions bot Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

5523 passed, 340 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

1 failed
❌ [installation tests] › playwright-cdn.spec.ts:43 › playwright cdn failover should work (https://cdn.playwright.dev/dbazure/download/playwright) @package-installations-windows-latest

9 flaky ⚠️ [installation tests] › playwright-cdn.spec.ts:43 › playwright cdn failover should work (https://playwright.download.prss.microsoft.com/dbazure/download/playwright) `@package-installations-ubuntu-latest`
⚠️ [installation tests] › playwright-cdn.spec.ts:43 › playwright cdn failover should work (https://playwright.download.prss.microsoft.com/dbazure/download/playwright) `@package-installations-windows-latest`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1223 › should display language-specific locators `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-page] › page/page-request-continue.spec.ts:754 › propagate headers cross origin redirect after interception `@chromium-ubuntu-22.04-node24`
⚠️ [firefox-library] › library/browsertype-connect.spec.ts:758 › run-server › should upload a folder `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/trace-viewer.spec.ts:1262 › should update highlight when typing locator `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-wait-for-function.spec.ts:104 › should work with strict CSP policy `@firefox-ubuntu-22.04-node20`
⚠️ [webkit-library] › library/screencast.spec.ts:75 › start reuses existing screencast when video recording is running `@webkit-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-test-output.spec.ts:118 › should collapse repeated console messages for test `@macos-latest-node20`

38895 passed, 845 skipped


Merge workflow run.

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