[Feature]: `Update snapshot` sticky header in UI Mode Attachments panel

Maintainer thường phản hồi trong vòng 1 ngày

Đánh giá

Issue này chưa được đánh giá.

Mô tả

🚀 Feature Request

Add an Update snapshot action to each IMAGE DIFF section in UI Mode's Attachments panel.

The existing IMAGE DIFF header would remain sticky while scrolling through the comparison, with Update snapshot replacing that screenshot's expected snapshot with the actual screenshot.

This follows existing UI Mode patterns where actions are placed in a sticky header for the content they operate on, for example the Source panel actions:

https://github.com/user-attachments/assets/8d2c998b-2e56-4132-bccc-d3b23305bb1c

Example

After a failed test run with failing snapshots:

1. Review a failed screenshot using the existing Diff / Actual / Expected views
2. Click Update snapshot for any intentional change which should have the current snapshot replaced by the Actual snapshot
3. Continue reviewing the remaining image diffs individually

The existing Attachments panel:

The proposed action would be added to the existing IMAGE DIFF header:

IMAGE DIFF                                         Update snapshot
──────────────────────────────────────────────────────────────────
           Diff  Actual  Expected  Side by side  Slider

The header would remain sticky while scrolling through a large comparison. The action only appears for IMAGE DIFF sections, not other attachments.

Alternative designs can be seen in prior art:

• https://github.com/abhivaikar/pw-ui-review

Motivation

When several screenshot assertions fail, UI Mode lets you review each diff, but not update an intentional change directly from that diff.

I'm aware that Playwright v1.56+ supports updateSnapshots in UI Mode (issue #37007 + PR #37098), but the current workflow has a few drawbacks:

1. If users enable it before running, snapshots are updated before users know whether the changes are correct
2. If users run tests normally to failure and review first, accepting a change requires enabling it and running the test again, which can be slow, flaky, and interrupts the review flow
3. Because updating is run-level, other changed snapshots can be updated before users have reviewed them

Chuẩn bị môi trường

Bắt đầu từ đâu

1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
3. Fork repository và làm thay đổi trên một nhánh.
4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của microsoft/playwright

• [Bug]: Uncaught TypeError in captureSecurityDetails when APIRequestContext reuses a destroyed keep-alive socket

  Có thể đã có người làm @microsoft-playwright-automation đã nhận 1 ngày trước. Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy
  needs-triage

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100

  microsoft/playwright#43728 · 3 bình luận · Mở ngày 28/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• [Feature]: Set `this.name` in `TargetClosedError` constructor

  Có thể làm lại được @LeonxLJX đã nhận 55 ngày trước và không có pull request nào đang mở. Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy
  P3-collecting-feedback

  Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100

  microsoft/playwright#42522 · 7 bình luận · Mở ngày 2/9/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• test.fail() in after hooks relabels an already-executed test and can make a failing run exit 0

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy

  Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100

  microsoft/playwright#43738 · Mở ngày 28/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• [Bug]: connectOverCDP({ noDefaults: true }) still overrides font families (Page.setFontFamilies) on headless browsers

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 42/100

  microsoft/playwright#43715 · Mở ngày 28/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• [Bug]: WebKit's injected PublicKeyCredential is a plain object, so WebAuthn libraries report no support

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy

  Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100

  microsoft/playwright#43711 · 1 bình luận · Mở ngày 28/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

Tất cả issue của microsoft/playwright

Issue tương tự

• Shell: serve the home app's Cookie Policy on a hard reload

  Có thể đã có người làm @2001anshkaushik đã nhận hôm nay. Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy
  feature module:ai

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100

  rocketride-org/rocketride-server#2899 · 1 bình luận · Mở ngày 29/10/2026

  Maintainer thường phản hồi trong vòng 2 ngày

• [Dashboard] [Threat Hunting] Document Details clips values at 375 px with no usable horizontal scroll

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100

  wazuh/wazuh-dashboard-plugins#9551 · Mở ngày 29/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• Parity scripts use python3 from PATH and silently skip every story without the package venv

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100

  gofish-graphics/gofish-graphics#1252 · Mở ngày 29/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• ceo-inbox: triage hourly instead of every 15 minutes

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy
  optimization P3 size:XS

  Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100

  josephfung/curia#2366 · Mở ngày 29/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

• stranded-follow-up sweep trusts any issue whose body carries the marker as the tracking issue

  Đang mở

  ⋯Báo cáo vấn đề với issue này

  • Có gì không ổn?

  • Đã có người nhận
  • Không phù hợp cho người mới
  • Đã cũ hoặc không hợp lệ
  • Vấn đề khác…
  • Gửi Hủy
  type:bug

  Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100

  nightgauge/nightgauge#2688 · Mở ngày 29/10/2026

  Maintainer thường phản hồi trong vòng 1 ngày

Thêm issue về TypeScript