From 3609b80d27bded28cd42f3b86b61ad656ae32e40 Mon Sep 17 00:00:00 2001 From: Forest Savage Date: Sun, 23 Aug 2026 22:55:23 -0800 Subject: [PATCH 1/3] Hover unblurs a blurred video the same way as an image Signed-off-by: Forest Savage --- CHANGELOG.md | 1 + .../ManualReviewJobContentBlurableVideo.tsx | 7 +++++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index eff079915..a83deb66c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,7 @@ ## Review Console +- Fixed hovering a blurred video not lifting blur the way images already do; the player now unblurs on hover without requiring play (#524) - Fixed a "Job submission failed" error that prevented reviewers from clearing jobs whose item had an unparseable `Created At` value; the decision now records instead of failing (#913) - Fixed a "Something Went Wrong" page when opening a job or queue whose `Created At` value was unparseable; the affected date now shows `Unknown` instead of blanking the view (#916) - Fixed "Oldest Task Age" on the MRT queues dashboard showing the newest job's age instead of the oldest (#909) diff --git a/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.tsx b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.tsx index e874a5976..2b40bf587 100644 --- a/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.tsx +++ b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.tsx @@ -65,7 +65,10 @@ export default function ManualReviewJobContentBlurableVideo(props: { }); return ( -
+
Date: Sun, 23 Aug 2026 23:31:50 -0800 Subject: [PATCH 2/3] Add a dest check for video hover unblur Signed-off-by: Forest Savage --- ...nualReviewJobContentBlurableVideo.test.tsx | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx diff --git a/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx new file mode 100644 index 000000000..64ed75b85 --- /dev/null +++ b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx @@ -0,0 +1,31 @@ +import { render } from '@testing-library/react'; +import React from 'react'; + +import '@testing-library/jest-dom/extend-expect'; + +import ManualReviewJobContentBlurableVideo from './ManualReviewJobContentBlurableVideo'; + +vi.mock('react-player', () => ({ + default: function MockPlayer() { + return
; + }, +})); + +describe('ManualReviewJobContentBlurableVideo hover unblur', () => { + it('keeps play-to-unblur and adds group-hover unblur on a blurred video', () => { + const { container } = render( + , + ); + + const wrapper = container.firstElementChild; + expect(wrapper?.className).toMatch(/\bgroup\b/); + + const blurred = wrapper?.querySelector('.blur-sm'); + expect(blurred).toBeTruthy(); + expect(blurred?.className).toMatch(/group-hover:blur-none/); + expect(blurred?.className).not.toMatch(/\bblur-0\b/); + }); +}); From cd0d42a5e73b7f294c9650c25564956258f38b76 Mon Sep 17 00:00:00 2001 From: Forest Savage <96553407+forest-savage1234@users.noreply.github.com> Date: Mon, 24 Aug 2026 02:33:09 -0800 Subject: [PATCH 3/3] Use the configured alias in the video hover test Signed-off-by: Forest Savage Co-Authored-By: OpenAI Codex --- .../ManualReviewJobContentBlurableVideo.test.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx index 64ed75b85..378049dac 100644 --- a/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx +++ b/client/src/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo.test.tsx @@ -3,7 +3,7 @@ import React from 'react'; import '@testing-library/jest-dom/extend-expect'; -import ManualReviewJobContentBlurableVideo from './ManualReviewJobContentBlurableVideo'; +import ManualReviewJobContentBlurableVideo from '@/webpages/dashboard/mrt/manual_review_job/ManualReviewJobContentBlurableVideo'; vi.mock('react-player', () => ({ default: function MockPlayer() {