Skip to content

Commit 34bd197

Browse files
authored
fix(report): remove redundant before-calling screenshot (#2512)
The before-calling screenshot in the report detail panel is derived from task.uiContext.screenshot. It is the same image as UI Context, not a separately captured recorder snapshot. The TaskRunner can also reuse UI Context within its 300ms cache window. That makes the before-calling label ambiguous, so keep UI Context and remove the duplicate image.
1 parent 772272a commit 34bd197

1 file changed

Lines changed: 0 additions & 11 deletions

File tree

  • apps/report/src/components/detail-panel

apps/report/src/components/detail-panel/index.tsx

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -249,17 +249,6 @@ const DetailPanel = (): JSX.Element => {
249249
</ScreenshotDisplay>
250250
) : null;
251251

252-
// screenshot view
253-
const screenshotFromContext = activeTask.uiContext?.screenshot;
254-
if (screenshotFromContext?.base64) {
255-
screenshotItems.push({
256-
timestamp: activeTask.timing?.start ?? undefined,
257-
screenshotTimestamp: screenshotFromContext.capturedAt,
258-
screenshot: screenshotFromContext.base64,
259-
timing: 'before-calling',
260-
});
261-
}
262-
263252
if (activeTask.recorder?.length) {
264253
for (const item of activeTask.recorder) {
265254
if (item.screenshot?.base64) {

0 commit comments

Comments
 (0)