Merge pull request #462 from creative-commoners/pulls/4.0/show-placeh…

…older

ENH Add placeholder when no summary content available

client/src/components/ElementEditor/Summary.js

@@ -1,8 +1,21 @@
 import React, { PureComponent, PropTypes } from 'react';
+import i18n from 'i18n';
 
 class Summary extends PureComponent {
+  /**
+   * Renders a preview of the Element's content, allowing for either a simple
+   * image, or a section of text content (or both). For an Element with only a
+   * file, we do NOT wish to render the "no preview available" filler content.
+   * Elements with content and/or a file should always show the text and/or
+   * the image.
+   *  - if there is a file, show that.
+   *  - if there is text, show that.
+   *  - if there is a file and text show both.
+   *  - if there is no file, and no text, substitute and show "no preview" text.
+   */
   render() {
     const { fileUrl, fileTitle, content } = this.props;
+    const noContent = i18n._t('Summary.NO_PREVIEW', 'No preview available');
 
     return (
 
@@ -16,7 +29,7 @@ class Summary extends PureComponent {
         }
         {(content || !fileUrl) &&
         <p className="element-editor-summary__content">
-          {content}
+          {content || noContent}
         </p>
         }
       </div>
@@ -34,4 +47,3 @@ Summary.propTypes = {
 };
 
 export default Summary;
-