From ecfdd2842d7f795d644c2b20266f0b3d04b849b5 Mon Sep 17 00:00:00 2001 From: "jake.oeding" Date: Mon, 3 Apr 2023 15:18:44 -0400 Subject: [PATCH] update studio media options tray copy closes MAT-1309 flag=rce_show_studio_media_options test plan: -embed a studio video in the RCE via the lti tool -make sure to uncheck 'display media tabs' >open the tray and confirm it has the updated copy qa risk: low Change-Id: I778c426026024b0cd84f08606ef637540945012d Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/314988 Tested-by: Service Cloud Jenkins Reviewed-by: Yona Appletree QA-Review: Yona Appletree Product-Review: Allison Howell --- .../__tests__/VideoOptionsTray.test.js | 10 +++++----- .../instructure_record/VideoOptionsTray/index.js | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/__tests__/VideoOptionsTray.test.js b/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/__tests__/VideoOptionsTray.test.js index 9356b456986..4e6d16b30e6 100644 --- a/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/__tests__/VideoOptionsTray.test.js +++ b/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/__tests__/VideoOptionsTray.test.js @@ -244,7 +244,7 @@ describe('RCE "Videos" Plugin > VideoOptionsTray', () => { }) }) - describe('Studio Options Tray', () => { + describe('Studio Media Options Tray', () => { beforeEach(() => { props.studioOptions = { resizable: true, @@ -252,14 +252,14 @@ describe('RCE "Videos" Plugin > VideoOptionsTray', () => { } }) - it('is labeled with "Canvas Studio Options Tray"', () => { + it('is labeled correctly', () => { const {getByLabelText} = render() - expect(getByLabelText('Canvas Studio Options Tray')).toBeInTheDocument() + expect(getByLabelText('Studio Media Options Tray')).toBeInTheDocument() }) - it('the heading says "Canvas Studio Options"', () => { + it('has the correct heading', () => { const {getByText} = render() - expect(getByText('Canvas Studio Options')).toBeInTheDocument() + expect(getByText('Studio Media Options')).toBeInTheDocument() }) it('has a "Media Title" field', () => { diff --git a/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/index.js b/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/index.js index 45ce2db8b5e..ae506845eb4 100644 --- a/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/index.js +++ b/packages/canvas-rce/src/rce/plugins/instructure_record/VideoOptionsTray/index.js @@ -160,7 +160,7 @@ export default function VideoOptionsTray({ data-mce-component={true} label={ isStudio - ? formatMessage('Canvas Studio Options Tray') + ? formatMessage('Studio Media Options Tray') : formatMessage('Video Options Tray') } mountNode={instuiPopupMountNode} @@ -180,7 +180,7 @@ export default function VideoOptionsTray({ {isStudio - ? formatMessage('Canvas Studio Options') + ? formatMessage('Studio Media Options') : formatMessage('Video Options')}