Skip to content

Conversation

@cpAdm
Copy link
Contributor

@cpAdm cpAdm commented Mar 26, 2025

Moves the copy request buttons to a dropdown on the toolbar:
image

Says 'copied' when option is pressed
image

references: #35214

@github-actions

This comment has been minimized.

@cpAdm
Copy link
Contributor Author

cpAdm commented Mar 26, 2025

Failing test seems flaky to me?

Copy link
Contributor

@dgozman dgozman left a comment

Choose a reason for hiding this comment

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

@cpAdm Thank you for the PR! This is much more manageable to review. I left a few comments, but overall looks good.

@cpAdm cpAdm requested a review from dgozman March 28, 2025 13:50
@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › tests/reporter-html.spec.ts:2765:5 › created › should include diff in AI prompt @ubuntu-latest-node18-2

2 flaky ⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:424:3 › should throw for too deep reference chain @firefox-ubuntu-22.04-node18
⚠️ [webkit-library] › tests/library/trace-viewer.spec.ts:1306:1 › should pick locator in iframe @webkit-ubuntu-22.04-node18

38936 passed, 805 skipped
✔️✔️✔️

Merge workflow run.

@cpAdm
Copy link
Contributor Author

cpAdm commented Mar 28, 2025

Again, failed test seems flaky

Copy link
Member

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

Change looks good! Left one small comment.

@cpAdm cpAdm requested a review from Skn0tt March 31, 2025 12:48
Copy link
Member

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

@dgozman for final review

Copy link
Contributor

@dgozman dgozman left a comment

Choose a reason for hiding this comment

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

Looks good, merging in. Thank you for the PR!

@dgozman dgozman merged commit aa278d3 into microsoft:main Mar 31, 2025
29 checks passed
@cpAdm cpAdm deleted the feat-move-copy-request-buttons-to-toolbar branch March 31, 2025 13:22
@github-actions
Copy link
Contributor

Test results for "tests 1"

2 flaky ⚠️ [firefox-page] › tests/page/page-evaluate.spec.ts:424:3 › should throw for too deep reference chain @firefox-ubuntu-22.04-node18
⚠️ [webkit-page] › tests/page/page-screenshot.spec.ts:345:5 › page screenshot › should work while navigating @webkit-ubuntu-22.04-node18

38937 passed, 805 skipped
✔️✔️✔️

Merge workflow run.

whazor pushed a commit to whazor/playwright-trace-viewer-plus that referenced this pull request Jan 10, 2026
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