From c4a7b97b632020b94451d3addd49da826a4c22dc Mon Sep 17 00:00:00 2001
From: Ken Clary <kclary@edx.org>
Date: Wed, 30 Nov 2022 12:36:42 -0500
Subject: [PATCH] fix: correct various strings, including making them
 sentence-case. TNL-10240.

---
 .../EditorFooter/__snapshots__/index.test.jsx.snap   | 12 ++++++------
 .../components/EditorFooter/messages.js              |  4 ++--
 .../__snapshots__/index.test.jsx.snap                |  4 ++--
 .../components/ThumbnailWidget/messages.js           |  2 +-
 .../__snapshots__/Transcript.test.jsx.snap           |  2 +-
 .../__snapshots__/index.test.jsx.snap                | 12 ++++++------
 .../components/TranscriptWidget/index.jsx            |  2 +-
 .../components/TranscriptWidget/messages.js          |  7 ++++++-
 .../__snapshots__/index.test.jsx.snap                |  4 ++--
 .../components/VideoSourceWidget/messages.js         |  4 ++--
 10 files changed, 29 insertions(+), 24 deletions(-)

diff --git a/src/editors/containers/EditorContainer/components/EditorFooter/__snapshots__/index.test.jsx.snap b/src/editors/containers/EditorContainer/components/EditorFooter/__snapshots__/index.test.jsx.snap
index 5011cdcd7..5bbecff01 100644
--- a/src/editors/containers/EditorContainer/components/EditorFooter/__snapshots__/index.test.jsx.snap
+++ b/src/editors/containers/EditorContainer/components/EditorFooter/__snapshots__/index.test.jsx.snap
@@ -16,7 +16,7 @@ exports[`EditorFooter render snapshot: default args (disableSave: false, saveFai
     <ActionRow>
       <ActionRow.Spacer />
       <Button
-        aria-label="Discard Changes and Return to Learning Context"
+        aria-label="Discard changes and return to learning context"
         onClick={[MockFunction args.onCancel]}
         variant="tertiary"
       >
@@ -27,7 +27,7 @@ exports[`EditorFooter render snapshot: default args (disableSave: false, saveFai
         />
       </Button>
       <Button
-        aria-label="Save Changes and Return to Learning Context"
+        aria-label="Save changes and return to learning context"
         disabled={false}
         onClick={[MockFunction args.onSave]}
       >
@@ -58,7 +58,7 @@ exports[`EditorFooter render snapshot: save disabled. Show button spinner 1`] =
     <ActionRow>
       <ActionRow.Spacer />
       <Button
-        aria-label="Discard Changes and Return to Learning Context"
+        aria-label="Discard changes and return to learning context"
         onClick={[MockFunction args.onCancel]}
         variant="tertiary"
       >
@@ -69,7 +69,7 @@ exports[`EditorFooter render snapshot: save disabled. Show button spinner 1`] =
         />
       </Button>
       <Button
-        aria-label="Save Changes and Return to Learning Context"
+        aria-label="Save changes and return to learning context"
         disabled={true}
         onClick={[MockFunction args.onSave]}
       >
@@ -109,7 +109,7 @@ exports[`EditorFooter render snapshot: save failed.  Show error message 1`] = `
     <ActionRow>
       <ActionRow.Spacer />
       <Button
-        aria-label="Discard Changes and Return to Learning Context"
+        aria-label="Discard changes and return to learning context"
         onClick={[MockFunction args.onCancel]}
         variant="tertiary"
       >
@@ -120,7 +120,7 @@ exports[`EditorFooter render snapshot: save failed.  Show error message 1`] = `
         />
       </Button>
       <Button
-        aria-label="Save Changes and Return to Learning Context"
+        aria-label="Save changes and return to learning context"
         disabled={false}
         onClick={[MockFunction args.onSave]}
       >
diff --git a/src/editors/containers/EditorContainer/components/EditorFooter/messages.js b/src/editors/containers/EditorContainer/components/EditorFooter/messages.js
index 26995d139..6058d2960 100644
--- a/src/editors/containers/EditorContainer/components/EditorFooter/messages.js
+++ b/src/editors/containers/EditorContainer/components/EditorFooter/messages.js
@@ -6,7 +6,7 @@ export const messages = {
   },
   cancelButtonAriaLabel: {
     id: 'authoring.editorfooter.cancelButton.ariaLabel',
-    defaultMessage: 'Discard Changes and Return to Learning Context',
+    defaultMessage: 'Discard changes and return to learning context',
     description: 'Screen reader label for cancel button',
   },
   cancelButtonLabel: {
@@ -16,7 +16,7 @@ export const messages = {
   },
   saveButtonAriaLabel: {
     id: 'authoring.editorfooter.savebutton.ariaLabel',
-    defaultMessage: 'Save Changes and Return to Learning Context',
+    defaultMessage: 'Save changes and return to learning context',
     description: 'Screen reader label for save button',
   },
   saveButtonLabel: {
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/__snapshots__/index.test.jsx.snap
index b38f3025e..fd29f9eed 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/__snapshots__/index.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/__snapshots__/index.test.jsx.snap
@@ -138,7 +138,7 @@ exports[`ThumbnailWidget snapshots snapshots: renders as expected where videoTyp
       variant="link"
     >
       <FormattedMessage
-        defaultMessage="Upload Thumbnail"
+        defaultMessage="Upload thumbnail"
         description="Label for upload button"
         id="authoring.videoeditor.thumbnail.upload.label"
       />
@@ -246,7 +246,7 @@ exports[`ThumbnailWidget snapshots snapshots: renders as expected with default p
       variant="link"
     >
       <FormattedMessage
-        defaultMessage="Upload Thumbnail"
+        defaultMessage="Upload thumbnail"
         description="Label for upload button"
         id="authoring.videoeditor.thumbnail.upload.label"
       />
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/messages.js b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/messages.js
index 0d65283c6..1b65e4d85 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/messages.js
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/ThumbnailWidget/messages.js
@@ -26,7 +26,7 @@ export const messages = {
   },
   uploadButtonLabel: {
     id: 'authoring.videoeditor.thumbnail.upload.label',
-    defaultMessage: 'Upload Thumbnail',
+    defaultMessage: 'Upload thumbnail',
     description: 'Label for upload button',
   },
   addThumbnail: {
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/Transcript.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/Transcript.test.jsx.snap
index 832e9b51b..e6dbdca5b 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/Transcript.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/Transcript.test.jsx.snap
@@ -40,7 +40,7 @@ exports[`Transcript Component component component snapshots: renders as expected
     <Card.Header
       title={
         <FormattedMessage
-          defaultMessage="Delete This Transcript?"
+          defaultMessage="Delete this transcript?"
           description="Title for Warning which allows users to select next step in the process of deleting a transcript"
           id="authoring.videoeditor.transcripts.deleteConfirmationTitle"
         />
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/index.test.jsx.snap
index 69c8f379e..1484dd982 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/index.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/__snapshots__/index.test.jsx.snap
@@ -4,7 +4,7 @@ exports[`TranscriptWidget component snapshots snapshot: renders ErrorAlert with
 <CollapsibleFormWidget
   isError={true}
   subtitle="English"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
@@ -102,7 +102,7 @@ exports[`TranscriptWidget component snapshots snapshot: renders ErrorAlert with
 <CollapsibleFormWidget
   isError={true}
   subtitle="English, French"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
@@ -204,7 +204,7 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit
 <CollapsibleFormWidget
   isError={true}
   subtitle="English"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
@@ -302,7 +302,7 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit
 <CollapsibleFormWidget
   isError={true}
   subtitle="None"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
@@ -366,7 +366,7 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit
 <CollapsibleFormWidget
   isError={true}
   subtitle="English"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
@@ -464,7 +464,7 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit
 <CollapsibleFormWidget
   isError={true}
   subtitle="English"
-  title="Transcript"
+  title="Transcripts"
 >
   <ErrorAlert
     dismissError={null}
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
index 5dbe1a307..3e85a603a 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
@@ -92,7 +92,7 @@ export const TranscriptWidget = ({
     <CollapsibleFormWidget
       isError={Object.keys(error).length !== 0}
       subtitle={fullTextLanguages}
-      title="Transcript"
+      title={intl.formatMessage(messages.title)}
     >
       <ErrorAlert
         hideHeading
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/messages.js b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/messages.js
index 64c0246e9..5089673c0 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/messages.js
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/messages.js
@@ -1,4 +1,9 @@
 export const messages = {
+  title: {
+    id: 'authoring.videoeditor.transcripts.title',
+    defaultMessage: 'Transcripts',
+    description: 'Title for transcripts widget',
+  },
   uploadButtonLabel: {
     id: 'authoring.videoeditor.transcripts.upload.label',
     defaultMessage: 'Add a transcript',
@@ -81,7 +86,7 @@ export const messages = {
   },
   deleteConfirmationHeader: {
     id: 'authoring.videoeditor.transcripts.deleteConfirmationTitle',
-    defaultMessage: 'Delete This Transcript?',
+    defaultMessage: 'Delete this transcript?',
     description: 'Title for Warning which allows users to select next step in the process of deleting a transcript',
   },
   fileTypeWarning: {
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap
index 555c808cf..b878b9fe8 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/__snapshots__/index.test.jsx.snap
@@ -2,7 +2,7 @@
 
 exports[`VideoSourceWidget snapshots snapshots: renders as expected with default props 1`] = `
 <injectIntl(ShimmedIntlComponent)
-  title="Video Source"
+  title="Video source"
 >
   <Form.Group>
     <div
@@ -19,7 +19,7 @@ exports[`VideoSourceWidget snapshots snapshots: renders as expected with default
       className="mt-3"
     >
       <FormattedMessage
-        defaultMessage="Fallback Videos"
+        defaultMessage="Fallback videos"
         description="Title for the fallback videos section"
         id="authoring.videoeditor.videoSource.fallbackVideo.title"
       />
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/messages.js b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/messages.js
index 640a1dbb9..27df6fc0d 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/messages.js
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/VideoSourceWidget/messages.js
@@ -1,7 +1,7 @@
 export const messages = {
   titleLabel: {
     id: 'authoring.videoeditor.videoSource.title.label',
-    defaultMessage: 'Video Source',
+    defaultMessage: 'Video source',
     description: 'Title for the video source widget',
   },
   videoIdOrUrlLabel: {
@@ -17,7 +17,7 @@ export const messages = {
   },
   fallbackVideoTitle: {
     id: 'authoring.videoeditor.videoSource.fallbackVideo.title',
-    defaultMessage: 'Fallback Videos',
+    defaultMessage: 'Fallback videos',
     description: 'Title for the fallback videos section',
   },
   fallbackVideoMessage: {