diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/CollapsibleFormWidget.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/CollapsibleFormWidget.jsx index 1ff7caf1d..679c4530b 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/CollapsibleFormWidget.jsx +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/CollapsibleFormWidget.jsx @@ -24,7 +24,7 @@ export const CollapsibleFormWidget = ({ intl, }) => ( @@ -32,17 +32,17 @@ export const CollapsibleFormWidget = ({ className="collapsible-trigger d-flex border-0 align-items-center" style={{ justifyContent: 'unset' }} > - +
{title}
-
{subtitle}
+ {subtitle ?
{subtitle}
: null}
{isError && }
- +
{title}
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/DurationWidget.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/DurationWidget.jsx index 4c875aedf..779266af6 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/DurationWidget.jsx +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/DurationWidget.jsx @@ -46,7 +46,7 @@ export const DurationWidget = ({ subtitle={getTotalLabel(duration.formValue.startTime, duration.formValue.stopTime, true)} > - + ( level !== LicenseLevel.course && details && license !== 'select' ? (
- -
+ +
- {license === LicenseTypes.allRightsReserved - ? ( -
- -
- ) - : null} + {license === LicenseTypes.allRightsReserved ? ( + + ) : null} {license === LicenseTypes.creativeCommons ? ( - -
+ +
@@ -67,7 +63,7 @@ export const LicenseDetails = ({
-
+
@@ -92,7 +88,7 @@ export const LicenseDetails = ({
-
+
@@ -145,9 +141,7 @@ export const LicenseDetails = ({
- ) - : null} - + ) : null}
) : null diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx index 9fcfd067e..2cf92a661 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx @@ -24,9 +24,9 @@ export const LicenseDisplay = ({ return (
-
+
-
{licenseDescription}
+
{licenseDescription}
) : null } -
{levelDescription}
- {license === LicenseTypes.select ? null :
} +
{levelDescription}
+ {license === LicenseTypes.select ? null :
} ); }; diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDetails.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDetails.test.jsx.snap index 1db86ef43..63083c1e8 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDetails.test.jsx.snap +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseDetails.test.jsx.snap @@ -6,8 +6,12 @@ exports[`LicenseDetails snapshots snapshots: renders as expected with level set
- -
+ +
@@ -51,7 +55,7 @@ exports[`LicenseDetails snapshots snapshots: renders as expected with level set
@@ -82,7 +86,7 @@ exports[`LicenseDetails snapshots snapshots: renders as expected with level set
@@ -150,23 +154,23 @@ exports[`LicenseDetails snapshots snapshots: renders as expected with level set
- -
+ +
-
- -
+
`; @@ -177,8 +181,12 @@ exports[`LicenseDetails snapshots snapshots: renders as expected with level set
- -
+ +
FormattedMessage component with license description
@@ -54,14 +54,14 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with level set />
FormattedMessage component with license description
@@ -94,14 +94,14 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with level set />
FormattedMessage component with license description
@@ -136,14 +136,14 @@ exports[`LicenseDisplay snapshots snapshots: renders as expected with level set />
FormattedMessage component with license description
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseSelector.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseSelector.test.jsx.snap index 84a0f4383..4f2262ed2 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseSelector.test.jsx.snap +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/LicenseSelector.test.jsx.snap @@ -44,7 +44,7 @@ exports[`LicenseSelector snapshots snapshots: renders as expected with block lev />
`; @@ -90,7 +90,7 @@ exports[`LicenseSelector snapshots snapshots: renders as expected with default p
`; @@ -148,7 +148,7 @@ exports[`LicenseSelector snapshots snapshots: renders as expected with library l />
`; @@ -194,7 +194,7 @@ exports[`LicenseSelector snapshots snapshots: renders as expected with no licens
`; diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap index ea72233a5..494eaf4a2 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/__snapshots__/index.test.jsx.snap @@ -22,7 +22,7 @@ exports[`LicenseWidget snapshots snapshots: renders as expected with default pro title="License" >
- +
`; @@ -150,11 +147,9 @@ exports[`TranscriptWidget component snapshots snapshot: renders ErrorAlert with /> - +
- - +
`; @@ -274,17 +268,15 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit /> - +
- - +
`; @@ -394,16 +385,15 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit /> - - +
`; @@ -452,17 +442,15 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit /> - +
- - +
`; @@ -572,17 +559,15 @@ exports[`TranscriptWidget component snapshots snapshots: renders as expected wit /> - +
- - +
`; 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 5e111a7c8..ee3bf46f6 100644 --- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx +++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx @@ -107,17 +107,16 @@ export const TranscriptWidget = ({ > - + {hasTranscripts ? ( - - + { transcripts.map((language, index) => ( ))} - + updateField({ showTranscriptByDefault: e.target.checked })} >
@@ -155,8 +154,7 @@ export const TranscriptWidget = ({ )} - - +
- +
); 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 5cf0e7f3b..15b537b80 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 @@ -50,7 +50,9 @@ exports[`VideoSourceWidget snapshots snapshots: renders as expected with default id="authoring.videoeditor.videoSource.fallbackVideo.title" />
-
+
+
-
+
{fallbackVideos.formValue.length > 0 ? fallbackVideos.formValue.map((videoUrl, index) => ( - + )) : null} - + +