1) fix active Share alive checkbox; 2) show license details link only…

… for creative commons; 3) same color for Attribution blurb image
openedx · Jun 18, 2024 · 99cd9da · 99cd9da
1 parent 6a58779
commit 99cd9da
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/...iners/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseBlurb.jsx b/...iners/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseBlurb.jsx
@@ -25,7 +25,7 @@ export const LicenseBlurb = ({
     {/* not sure how to handle the edge cases when some of the icons are not displayed */}
     {license === LicenseTypes.allRightsReserved ? <Icon src={Copyright} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}
     {license === LicenseTypes.creativeCommons ? <Icon src={Cc} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}
-    {details.attribution ? <Icon src={Attribution} className="mr-1 text-primary-300" style={{ height: '18px', width: '18px' }} /> : null}
+    {details.attribution ? <Icon src={Attribution} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}
     {details.noncommercial ? <Icon src={Nc} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}
     {details.noDerivatives ? <Icon src={Nd} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}
     {details.shareAlike ? <Icon src={Sa} className="mr-1" style={{ height: '18px', width: '18px' }} /> : null}

diff --git a/...ers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx b/...ers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDetails.jsx
@@ -123,7 +123,7 @@ export const LicenseDetails = ({
                     </Form.Label>
                     <ActionRow.Spacer />
                     <CheckboxControl
-                      cchecked={details.shareAlike}
+                      checked={details.shareAlike}
                       disabled={level === LicenseLevel.course}
                       onChange={(e) => updateField({
                         licenseDetails: {

diff --git a/...ers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx b/...ers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseDisplay.jsx
@@ -28,13 +28,15 @@ export const LicenseDisplay = ({
           <LicenseBlurb license={license} details={details} />
           <div className="x-small mt-3">{licenseDescription}</div>
         </div>
-        <Hyperlink
-          className="text-primary-500 x-small"
-          destination="https://creativecommons.org/about"
-          target="_blank"
-        >
-          <FormattedMessage {...messages.viewLicenseDetailsLabel} />
-        </Hyperlink>
+        {license === LicenseTypes.creativeCommons && (
+          <Hyperlink
+            className="text-primary-500 x-small"
+            destination="https://creativecommons.org/about"
+            target="_blank"
+          >
+            <FormattedMessage {...messages.viewLicenseDetailsLabel} />
+          </Hyperlink>
+        )}
       </Stack>
     );
   }