learningtribes-studio-frontend 1.4.2-alpha.4 → 1.4.2-alpha.5

package/src/components/EditImageModal/EditImageModal.test.jsx

@@ -1,5 +1,5 @@
 import { Button, CheckBox, Fieldset, Modal, InputText, StatusAlert, Variant } from '@edx/paragon';
-import { IntlProvider, FormattedMessage } from 'react-intl';
+import { IntlProvider } from 'react-intl';
 import { Provider } from 'react-redux';
 import React from 'react';
 
@@ -41,7 +41,6 @@ const getInsertImageButton = editImageModal =>
   (getModalFooter(editImageModal).find(Button).first());
 const getNextButton = editImageModal =>
   (getModalFooter(editImageModal).find(Button).first());
-const getPreviousButton = editImageModal => (getModalBody(editImageModal).find(Button));
 
 const getImageDescriptionFieldset = editImageModal =>
   (getFormContainer(editImageModal).find(Fieldset).at(0));
@@ -159,10 +158,11 @@ describe('EditImageModal', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsDropZone)')).toHaveLength(1);
             });
 
-            it('a header', () => {
-              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('tagName') === 'h3');
-              expect(header).toHaveLength(1);
-              expect(header.prop('message')).toEqual(messages.editImageModalInsertHeader);
+            it('no insert header', () => {
+              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(
+                message => message.prop('message') === messages.editImageModalInsertHeader,
+              );
+              expect(header).toHaveLength(0);
             });
 
             it('a loading icon', () => {
@@ -218,30 +218,29 @@ describe('EditImageModal', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsDropZone)')).toHaveLength(1);
             });
 
-            it('a header', () => {
-              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('tagName') === 'h3');
-              expect(header).toHaveLength(1);
-              expect(header.prop('message')).toEqual(messages.editImageModalInsertHeader);
+            it('no insert header', () => {
+              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('message') === messages.editImageModalInsertHeader);
+              expect(header).toHaveLength(0);
             });
 
             it('an AssetsList component', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsList)')).toHaveLength(1);
             });
 
-            it('an AssetsSearch component', () => {
-              expect(getModalBody(editImageModal).find('Connect(AssetsSearch)')).toHaveLength(1);
+            it('no AssetsSearch component', () => {
+              expect(getModalBody(editImageModal).find('Connect(AssetsSearch)')).toHaveLength(0);
             });
 
             it('no AssetsClearSearchButton component', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsClearSearchButton)')).toHaveLength(0);
             });
 
-            it('a AssetsResultsCount component', () => {
-              expect(getModalBody(editImageModal).find('Connect(AssetsResultsCount)')).toHaveLength(1);
+            it('no AssetsResultsCount component', () => {
+              expect(getModalBody(editImageModal).find('Connect(AssetsResultsCount)')).toHaveLength(0);
             });
 
-            it('a Pagination component', () => {
-              expect(getModalBody(editImageModal).find('Connect(InjectIntl(Pagination))')).toHaveLength(1);
+            it('no Pagination component', () => {
+              expect(getModalBody(editImageModal).find('Connect(InjectIntl(Pagination))')).toHaveLength(0);
             });
           });
 
@@ -263,23 +262,26 @@ describe('EditImageModal', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsDropZone)')).toHaveLength(1);
             });
 
-            it('a header', () => {
-              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('tagName') === 'h3').at(0);
-              expect(header).toHaveLength(1);
-              expect(header.prop('message')).toEqual(messages.editImageModalInsertHeader);
+            it('no insert header', () => {
+              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(
+                message => message.prop('message') === messages.editImageModalInsertHeader,
+              );
+              expect(header).toHaveLength(0);
             });
 
             it('a no results message', () => {
-              expect(getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('tagName') === 'h3').at(1)
-                .prop('message')).toEqual(messages.editImageModalAssetsListNoResultsMessage);
+              const noResultsMessage = getModalBody(editImageModal).find(WrappedMessage).filterWhere(
+                message => message.prop('message') === messages.editImageModalAssetsListNoResultsMessage,
+              );
+              expect(noResultsMessage).toHaveLength(1);
             });
 
             it('no AssetsList component', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsList)')).toHaveLength(0);
             });
 
-            it('an AssetsSearch component', () => {
-              expect(getModalBody(editImageModal).find('Connect(AssetsSearch)')).toHaveLength(1);
+            it('no AssetsSearch component', () => {
+              expect(getModalBody(editImageModal).find('Connect(AssetsSearch)')).toHaveLength(0);
             });
 
             it('an AssetsClearSearchButton component', () => {
@@ -314,10 +316,11 @@ describe('EditImageModal', () => {
               expect(getModalBody(editImageModal).find('Connect(AssetsDropZone)'));
             });
 
-            it('a header', () => {
-              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(message => message.prop('tagName') === 'h3').at(0);
-              expect(header).toHaveLength(1);
-              expect(header.prop('message')).toEqual(messages.editImageModalInsertHeader);
+            it('no insert header', () => {
+              const header = getModalBody(editImageModal).find(WrappedMessage).filterWhere(
+                message => message.prop('message') === messages.editImageModalInsertHeader,
+              );
+              expect(header).toHaveLength(0);
             });
 
             it('no empty-state message', () => {
@@ -356,8 +359,7 @@ describe('EditImageModal', () => {
         beforeEach(() => {
           editImageModal.setState({
             open: true,
-            pageNumber: 2,
-            shouldShowPreviousButton: true,
+            imageSource: sampleText,
           });
         });
 
@@ -369,7 +371,7 @@ describe('EditImageModal', () => {
           const modal = getModal(editImageModal);
 
           expect(modal.prop('open')).toEqual(false);
-          expect(shallowWithIntl(modal.prop('title')).find(WrappedMessage).prop('message')).toEqual(messages.editImageModalEditTitle);
+          expect(shallowWithIntl(modal.prop('title')).find(WrappedMessage).prop('message')).toEqual(messages.editImageModalInsertTitle);
           expect(shallowWithIntl(modal.prop('closeText')).prop('message')).toEqual(messages.editImageModalCancelButton);
         });
 
@@ -390,30 +392,9 @@ describe('EditImageModal', () => {
 
           it('a StatusAlert component with correct initial props', () => {
             expect(getStatusAlert(editImageModal).prop('alertType')).toEqual('danger');
-            const statusAlertDialog = shallowWithIntl(getStatusAlert(editImageModal).prop('dialog')).find(WrappedMessage).at(0);
-            expect(statusAlertDialog.prop('message')).toEqual(messages.editImageModalFormErrorMissingFields);
             expect(getStatusAlert(editImageModal).prop('open')).toEqual(false);
           });
 
-          it('a previous page button when this.state.shouldShowPreviousButton', () => {
-            expect(getModalBody(editImageModal).find(Button)).toHaveLength(1);
-          });
-
-          it('a previous page button with correct props', () => {
-            const previousButton = getModalBody(editImageModal).find(Button);
-            expect(previousButton).toHaveLength(1);
-            expect(shallowWithIntl(previousButton.prop('label')).prop('message')).toEqual(messages.editImageModalPreviousPageButton);
-            expect(previousButton.prop('buttonType')).toEqual('link');
-          });
-
-          it('no previous page button when not this.state.shouldShowPreviousButton', () => {
-            editImageModal.setState({
-              shouldShowPreviousButton: false,
-            });
-
-            expect(getModalBody(editImageModal).find(Button)).toHaveLength(0);
-          });
-
           describe('a form with', () => {
             it('a form', () => {
               expect(getFormContainer(editImageModal)).toHaveLength(1);
@@ -594,8 +575,6 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isImageLoading')).toEqual(false);
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
         expect(editImageModal.state('isModalOpen')).toEqual(false);
-        expect(editImageModal.state('pageNumber')).toEqual(1);
-        expect(editImageModal.state('shouldShowPreviousButton')).toEqual(false);
       });
 
       it('handleOpenModal sets correct state for empty event (inserting an image) with assets', () => {
@@ -621,8 +600,6 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isImageLoading')).toEqual(false);
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
         expect(editImageModal.state('isModalOpen')).toEqual(true);
-        expect(editImageModal.state('pageNumber')).toEqual(1);
-        expect(editImageModal.state('shouldShowPreviousButton')).toEqual(true);
       });
 
       it('handleOpenModal sets correct state for empty event (inserting an image) without assets', () => {
@@ -644,8 +621,6 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isImageLoading')).toEqual(false);
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
         expect(editImageModal.state('isModalOpen')).toEqual(true);
-        expect(editImageModal.state('pageNumber')).toEqual(1);
-        expect(editImageModal.state('shouldShowPreviousButton')).toEqual(true);
       });
 
       it('handleOpenModal sets correct state for event with data (editing an image)', () => {
@@ -683,8 +658,7 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isImageLoading')).toEqual(false);
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
         expect(editImageModal.state('isModalOpen')).toEqual(true);
-        expect(editImageModal.state('pageNumber')).toEqual(2);
-        expect(editImageModal.state('shouldShowPreviousButton')).toEqual(false);
+        expect(editImageModal.state('editingExisting')).toEqual(true);
       });
 
       it('onEditImageModalClose sets this.state.open to false', () => {
@@ -704,11 +678,6 @@ describe('EditImageModal', () => {
       });
 
       it('onEditImageModalClose sets this.state.isModalOpen to false', () => {
-        editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
-
         const focusSpy = jest.fn();
         editImageModal.instance().setModalWrapperRef({
           dispatchEvent: jest.fn(),
@@ -790,11 +759,6 @@ describe('EditImageModal', () => {
 
         editImageModal.instance().setDropZoneButtonRef(refMock);
         editImageModal.instance().setStatusAlertRef(refMock);
-
-        editImageModal.setState({
-          pageNumber: 1,
-          shouldShowPreviousButton: true,
-        });
       });
 
       it('this.state.isStatusAlertOpen is false by default', () => {
@@ -881,13 +845,7 @@ describe('EditImageModal', () => {
           focus: jest.fn(),
         };
 
-        editImageModal.instance().setPreviousButtonRef(refMock);
         editImageModal.instance().setStatusAlertRef(refMock);
-
-        editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
       });
 
       it('this.state.isStatusAlertOpen is false by default', () => {
@@ -895,13 +853,27 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
       });
 
-      it('this.state.isStatusAlertOpen is true if invalid form submitted', () => {
+      it('this.state.isStatusAlertOpen is true if invalid dimensions are submitted', () => {
+        editImageModal.setState({
+          imageDimensions: {
+            height: -1 * sampleImgData.height,
+            width: -1 * sampleImgData.width,
+          },
+        });
+
         getInsertImageButton(editImageModal).simulate('click');
 
         expect(editImageModal.state('isStatusAlertOpen')).toEqual(true);
       });
 
-      it('status alert is displayed if invalid form submitted', () => {
+      it('status alert is displayed if invalid dimensions are submitted', () => {
+        editImageModal.setState({
+          imageDimensions: {
+            height: -1 * sampleImgData.height,
+            width: -1 * sampleImgData.width,
+          },
+        });
+
         getInsertImageButton(editImageModal).simulate('click');
 
         editImageModal.update();
@@ -910,6 +882,13 @@ describe('EditImageModal', () => {
       });
 
       it('StatusAlert open prop is false when status alert is closed', () => {
+        editImageModal.setState({
+          imageDimensions: {
+            height: -1 * sampleImgData.height,
+            width: -1 * sampleImgData.width,
+          },
+        });
+
         getInsertImageButton(editImageModal).simulate('click');
 
         getCloseStatusAlertButton(editImageModal).simulate('click');
@@ -958,6 +937,7 @@ describe('EditImageModal', () => {
 
       it('status alert has correct validation message when invalid dimensions submitted', () => {
         editImageModal.setState({
+          imageDescription: sampleText,
           imageDimensions: {
             height: (-1 * sampleImgData.naturalHeight),
             width: (-1 * sampleImgData.naturalWidth),
@@ -975,6 +955,7 @@ describe('EditImageModal', () => {
 
       it('validation messages are hyperlinks that navigate to appropriate form elements', () => {
         editImageModal.setState({
+          imageDescription: sampleText,
           imageDimensions: {
             height: (-1 * sampleImgData.naturalHeight),
             width: (-1 * sampleImgData.naturalWidth),
@@ -1009,13 +990,20 @@ describe('EditImageModal', () => {
         expect(statusAlertListItems.at(0).find(WrappedMessage).prop('message')).toEqual(messages.editImageModalFormValidImageDimensions);
       });
 
-      it('focus on page 2 is moved to previous button when status alert is closed', () => {
+      it('focuses the image description input when a validation alert is closed', () => {
         const focusSpy = jest.fn();
 
-        editImageModal.instance().setPreviousButtonRef({
+        editImageModal.instance().setImageDescriptionInputRef({
           focus: focusSpy,
         });
 
+        editImageModal.setState({
+          imageDimensions: {
+            height: -1 * sampleImgData.height,
+            width: -1 * sampleImgData.width,
+          },
+        });
+
         getInsertImageButton(editImageModal).simulate('click');
 
         getCloseStatusAlertButton(editImageModal).simulate('click');
@@ -1026,6 +1014,10 @@ describe('EditImageModal', () => {
       it('focus on page 2 is moved to image description input when status alert is closed and previous button is not shown', () => {
         editImageModal.setState({
           shouldShowPreviousButton: false,
+          imageDimensions: {
+            height: -1 * sampleImgData.height,
+            width: -1 * sampleImgData.width,
+          },
         });
 
         const focusSpy = jest.fn();
@@ -1050,8 +1042,7 @@ describe('EditImageModal', () => {
 
         editImageModal.setState({
           open: true,
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
+          imageSource: sampleText,
         });
       });
 
@@ -1101,13 +1092,7 @@ describe('EditImageModal', () => {
       };
 
       editImageModal.instance().setDropZoneButtonRef(refMock);
-      editImageModal.instance().setPreviousButtonRef(refMock);
       editImageModal.instance().setStatusAlertRef(refMock);
-
-      editImageModal.setState({
-        pageNumber: 1,
-        shouldShowPreviousButton: true,
-      });
     });
 
     it('state updated correctly on upload success', () => {
@@ -1131,7 +1116,7 @@ describe('EditImageModal', () => {
       expect(selectAssetSpy).toHaveBeenCalledTimes(1);
       expect(selectAssetSpy).toHaveBeenCalledWith(assetsStatusMock.response.asset, 0);
       expect(clearAssetsStatusSpy).toHaveBeenCalledTimes(1);
-      expect(editImageModal.state('pageNumber')).toEqual(2);
+      expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
     });
 
     describe('AssetsDropZone', () => {
@@ -1145,13 +1130,6 @@ describe('EditImageModal', () => {
     });
 
     describe('Image Preview', () => {
-      beforeEach(() => {
-        editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
-      });
-
       it('correct state is set on image load', () => {
         editImageModal.setState({
           imageSource: sampleText,
@@ -1181,57 +1159,6 @@ describe('EditImageModal', () => {
         expect(editImageModal.state('isImageLoaded')).toEqual(false);
         expect(editImageModal.state('isImageLoading')).toEqual(false);
       });
-
-      it('with visible image preview image when this.state.isImageLoaded is false (default)', () => {
-        editImageModal.setState({
-          imageSource: sampleText,
-        });
-
-        const img = getModalBody(editImageModal).find('img');
-
-        expect(img.prop('className')).toEqual(expect.stringContaining('invisible'));
-      });
-
-      it('with visible image preview image when this.state.isImageLoaded is true ', () => {
-        editImageModal.setState({
-          imageSource: sampleText,
-          isImageLoaded: true,
-        });
-
-        const img = getModalBody(editImageModal).find('img');
-
-        // TODO: replace this with asymmetric matcher expect.not.stringContaining when released
-        expect(img.prop('className').includes('invisible')).toEqual(false);
-      });
-
-      it('with visible image preview placeholder text when this.state.isImageLoaded is false (default)', () => {
-        const imagePreviewPlaceholderText = getModalBody(editImageModal).find('.image-preview-placeholder').find(WrappedMessage);
-
-        const imagePreviewPlaceholderTextMessage = imagePreviewPlaceholderText
-          .dive({ context: { store, intl } })
-          .dive({ context: { store, intl } }).find(FormattedMessage)
-          .dive({ context: { store, intl } });
-
-        // TODO: replace this with asymmetric matcher expect.not.stringContaining when released
-        expect(imagePreviewPlaceholderTextMessage.prop('className').includes('invisible')).toEqual(false);
-      });
-
-      it('with invisible image preview placeholder text when this.state.isImageLoaded is true', () => {
-        editImageModal.setState({
-          isImageLoaded: true,
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
-
-        const imagePreviewPlaceholderText = getModalBody(editImageModal).find('.image-preview-placeholder').find(WrappedMessage);
-
-        const imagePreviewPlaceholderTextMessage = imagePreviewPlaceholderText
-          .dive({ context: { store, intl } })
-          .dive({ context: { store, intl } }).find(FormattedMessage)
-          .dive({ context: { store, intl } });
-
-        expect(imagePreviewPlaceholderTextMessage.prop('className')).toEqual(expect.stringContaining('invisible'));
-      });
     });
 
     describe('Image Description Fieldset', () => {
@@ -1241,12 +1168,10 @@ describe('EditImageModal', () => {
         };
 
         editImageModal.instance().setImageDescriptionInputRef(refMock);
-        editImageModal.instance().setPreviousButtonRef(refMock);
         editImageModal.instance().setStatusAlertRef(refMock);
 
         editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
+          imageSource: sampleText,
         });
       });
 
@@ -1268,8 +1193,7 @@ describe('EditImageModal', () => {
     describe('Image Dimensions Fieldset', () => {
       beforeEach(() => {
         editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
+          imageSource: sampleText,
         });
       });
 
@@ -1348,8 +1272,6 @@ describe('EditImageModal', () => {
         beforeEach(() => {
           editImageModal.setState({
             imageSource: sampleText,
-            pageNumber: 2,
-            shouldShowPreviousButton: true,
           });
 
           getModalBody(editImageModal).find('img').simulate('load', { target: { ...sampleImgData } });
@@ -1381,8 +1303,6 @@ describe('EditImageModal', () => {
         beforeEach(() => {
           editImageModal.setState({
             imageSource: sampleText,
-            pageNumber: 2,
-            shouldShowPreviousButton: true,
           });
 
           getImageDimensionsCheckBox(editImageModal).simulate('change', false);
@@ -1414,54 +1334,32 @@ describe('EditImageModal', () => {
       });
     });
 
-    describe('Next button', () => {
+    describe('Save button', () => {
       beforeEach(() => {
-        editImageModal.setState({
-          pageNumber: 1,
-          shouldShowPreviousButton: true,
-        });
-
         editImageModal.setProps({
           selectedAsset: {
             portable_url: 'foo',
           },
         });
-        const refMock = {
-          focus: jest.fn(),
-        };
-
-        editImageModal.instance().setPreviousButtonRef(refMock);
-      });
 
-      it('goes to page 2', () => {
-        const clearAssetsStatusSpy = jest.fn();
-        editImageModal.setProps({
-          clearAssetsStatus: clearAssetsStatusSpy,
+        editImageModal.instance().setImageFormRef({
+          dispatchEvent: jest.fn(),
         });
-
-        getNextButton(editImageModal).simulate('click');
-        editImageModal.update();
-
-        expect(editImageModal.state('isStatusAlertOpen')).toEqual(false);
-        expect(editImageModal.state('pageNumber')).toEqual(2);
-        expect(editImageModal.state('uploadErrorMessageBody')).toBeUndefined();
-        expect(clearAssetsStatusSpy).toHaveBeenCalledTimes(1);
       });
 
-      it('focuses on Previous/Back button', () => {
-        const focusSpy = jest.fn();
-        editImageModal.instance().setPreviousButtonRef({
-          focus: focusSpy,
-        });
-
+      it('does not submit when the description is empty', () => {
         getNextButton(editImageModal).simulate('click');
         editImageModal.update();
 
-        editImageModal.instance().componentDidUpdate({}, { pageNumber: 1 });
-        expect(focusSpy).toHaveBeenCalledTimes(1);
+        expect(editImageModal.state('isModalOpen')).toEqual(false);
+        expect(editImageModal.state('isImageDescriptionValid')).toEqual(false);
+        expect(editImageModal.state('isStatusAlertOpen')).toEqual(true);
       });
 
-      it('closes statusAlert', () => {
+      it('closes the status alert when the form is valid', () => {
+        editImageModal.setState({
+          imageDescription: sampleText,
+        });
         editImageModal.setProps({
           assetsStatus: {
             type: assetActions.upload.UPLOAD_EXCEED_MAX_COUNT_ERROR,
@@ -1478,52 +1376,19 @@ describe('EditImageModal', () => {
       });
     });
 
-    describe('Previous/Back button', () => {
-      beforeEach(() => {
-        editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
-        const refMock = {
-          focus: jest.fn(),
-        };
-
-        editImageModal.instance().setDropZoneButtonRef(refMock);
-      });
-
-      it('goes to page 1', () => {
-        getPreviousButton(editImageModal).simulate('click');
-        editImageModal.update();
-
-        expect(editImageModal.state('pageNumber')).toEqual(1);
-      });
-
-      it('focuses on Upload button', () => {
-        const focusSpy = jest.fn();
-        editImageModal.instance().setDropZoneButtonRef({
-          focus: focusSpy,
-        });
-
-        getPreviousButton(editImageModal).simulate('click');
-        editImageModal.update();
-
-        editImageModal.instance().componentDidUpdate({}, { isModalOpen: true, pageNumber: 2 });
-        expect(focusSpy).toHaveBeenCalledTimes(1);
-      });
-    });
-
     describe('Insert Image button', () => {
       let validationMock;
       let dispatchEventSpy;
 
       beforeEach(() => {
-        editImageModal.setState({
-          pageNumber: 2,
-          shouldShowPreviousButton: true,
-        });
-
         dispatchEventSpy = jest.fn();
 
+        editImageModal.setProps({
+          selectedAsset: {
+            id: 'selected-image',
+          },
+        });
+
         editImageModal.instance().setImageFormRef({
           dispatchEvent: dispatchEventSpy,
         });

package/src/components/EditImageModal/displayMessages.jsx

@@ -33,7 +33,7 @@ const messages = defineMessages({
   },
   editImageModalFormValidImageDescription: {
     id: 'editImageModalFormValidImageDescription',
-    defaultMessage: 'Add an image description or specify that the image is decorative only.',
+    defaultMessage: 'Add an image description.',
     description: 'Error message for when an invalid image description is entered into the form',
   },
   editImageModalFormValidImageDimensions: {
@@ -51,11 +51,6 @@ const messages = defineMessages({
     defaultMessage: 'Image Description',
     description: 'Label for the text input box for an image\'s description',
   },
-  editImageModalImageDecorativeDescription: {
-    id: 'editImageModalImageDecorativeDescription',
-    defaultMessage: 'If this image is purely decorative, it does not require alternative text.',
-    description: 'Help text shown under the image description when alternative text is optional',
-  },
   editImageModalImageDescriptionLegend: {
     id: 'editImageModalImageDescriptionLegend',
     defaultMessage: 'Image Description',