Markdown Support

.gitignore

@@ -26,3 +26,4 @@ build/daemon.zip
 .vimrc
 
 package-lock.json
+ui/yarn.lock

CHANGELOG.md

@@ -10,14 +10,17 @@ Web UI version numbers should always match the corresponding version of LBRY App
 ### Added
   * Added option to release claim when deleting a file
   * Added transition to card hovers to smooth animation
+  * Support markdown makeup in claim description
+  *
 
 ### Changed
-  *
+  * Publishes now uses claims rather than files
   *
 
 ### Fixed
   * Fixed bug with download notice when switching window focus
-  *
+  * Fixed newly published files appearing twice
+  * Fixed unconfirmed published files missing channel name
 
 ### Deprecated
   *

ui/js/component/common.js

@@ -1,5 +1,7 @@
 import React from "react";
+import ReactDOMServer from "react-dom/server";
 import lbry from "../lbry.js";
+import ReactMarkdown from "react-markdown";
 
 //component/icon.js
 export class Icon extends React.PureComponent {

ui/js/component/fileCard/view.jsx

@@ -1,7 +1,12 @@
 import React from "react";
 import lbryuri from "lbryuri.js";
 import Link from "component/link";
-import { TruncatedText, Icon } from "component/common";
+import {
+  Thumbnail,
+  TruncatedText,
+  Icon,
+  TruncatedMarkdown,
+} from "component/common";
 import FilePrice from "component/filePrice";
 import UriIndicator from "component/uriIndicator";
 import NsfwOverlay from "component/nsfwOverlay";
@@ -94,7 +99,7 @@ class FileCard extends React.PureComponent {
                 style={{ backgroundImage: "url('" + metadata.thumbnail + "')" }}
               />}
             <div className="card__content card__subtext card__subtext--two-lines">
-              <TruncatedText lines={2}>{description}</TruncatedText>
+              <TruncatedMarkdown lines={2}>{description}</TruncatedMarkdown>
             </div>
           </Link>
         </div>

ui/js/component/form.js

@@ -1,8 +1,9 @@
 import React from "react";
 import FileSelector from "./file-selector.js";
-import { Icon } from "./common.js";
+import SimpleMDE from "react-simplemde-editor";
+import style from "react-simplemde-editor/dist/simplemde.min.css";
 
-var formFieldCounter = 0,
+let formFieldCounter = 0,
   formFieldFileSelectorTypes = ["file", "directory"],
   formFieldNestedLabelTypes = ["radio", "checkbox"];
 
@@ -24,6 +25,7 @@ export class FormField extends React.PureComponent {
     this._fieldRequiredText = __("This field is required");
     this._type = null;
     this._element = null;
+    this._extraElementProps = {};
 
     this.state = {
       isError: null,
@@ -38,6 +40,12 @@ export class FormField extends React.PureComponent {
     } else if (this.props.type == "text-number") {
       this._element = "input";
       this._type = "text";
+    } else if (this.props.type == "SimpleMDE") {
+      this._element = SimpleMDE;
+      this._type = "textarea";
+      this._extraElementProps.options = {
+        hideIcons: ["guide", "heading", "image", "fullscreen"],
+      };
     } else if (formFieldFileSelectorTypes.includes(this.props.type)) {
       this._element = "input";
       this._type = "hidden";
@@ -81,6 +89,8 @@ export class FormField extends React.PureComponent {
   getValue() {
     if (this.props.type == "checkbox") {
       return this.refs.field.checked;
+    } else if (this.props.type == "SimpleMDE") {
+      return this.refs.field.simplemde.value();
     } else {
       return this.refs.field.value;
     }
@@ -90,6 +100,10 @@ export class FormField extends React.PureComponent {
     return this.refs.field.options[this.refs.field.selectedIndex];
   }
 
+  getOptions() {
+    return this.refs.field.options;
+  }
+
   render() {
     // Pass all unhandled props to the field element
     const otherProps = Object.assign({}, this.props),
@@ -106,7 +120,6 @@ export class FormField extends React.PureComponent {
     delete otherProps.className;
     delete otherProps.postfix;
     delete otherProps.prefix;
-
     const element = (
       <this._element
         id={elementId}
@@ -122,6 +135,7 @@ export class FormField extends React.PureComponent {
           (isError ? "form-field__input--error" : "")
         }
         {...otherProps}
+        {...this._extraElementProps}
       >
         {this.props.children}
       </this._element>
@@ -220,6 +234,10 @@ export class FormRow extends React.PureComponent {
     return this.refs.field.getSelectedElement();
   }
 
+  getOptions() {
+    return this.refs.field.getOptions();
+  }
+
   focus() {
     this.refs.field.focus();
   }

ui/js/page/filePage/view.jsx

@@ -1,4 +1,5 @@
 import React from "react";
+import ReactMarkdown from "react-markdown";
 import lbry from "lbry.js";
 import lbryuri from "lbryuri.js";
 import Video from "component/video";
@@ -119,7 +120,11 @@ class FilePage extends React.PureComponent {
               </div>
             </div>
             <div className="card__content card__subtext card__subtext card__subtext--allow-newlines">
-              {metadata && metadata.description}
+              <ReactMarkdown
+                source={(metadata && metadata.description) || ""}
+                escapeHtml={true}
+                disallowedTypes={["Heading", "HtmlInline", "HtmlBlock"]}
+              />
             </div>
           </div>
           {metadata

ui/js/page/publish/view.jsx

@@ -5,13 +5,16 @@ import { FormField, FormRow } from "component/form.js";
 import Link from "component/link";
 import rewards from "rewards";
 import Modal from "component/modal";
+import Notice from "component/notice";
 import { BusyMessage } from "component/common";
 
 class PublishPage extends React.PureComponent {
   constructor(props) {
     super(props);
 
-    this._requiredFields = ["meta_title", "name", "bid", "tos_agree"];
+    this._requiredFields = ["name", "bid", "meta_title", "tosAgree"];
+
+    this._defaultCopyrightNotice = "All rights reserved.";
 
     this.state = {
       rawName: "",
@@ -23,11 +26,17 @@ class PublishPage extends React.PureComponent {
       channel: "anonymous",
       newChannelName: "@",
       newChannelBid: 10,
-      myClaimValue: 0.0,
+      meta_title: "",
-      myClaimMetadata: null,
+      meta_thumbnail: "",
-      copyrightNotice: "",
+      meta_description: "",
+      meta_language: "en",
+      meta_nsfw: "0",
+      licenseType: "",
+      copyrightNotice: this._defaultCopyrightNotice,
       otherLicenseDescription: "",
       otherLicenseUrl: "",
+      tosAgree: false,
+      prefillDone: false,
       uploadProgress: 0.0,
       uploaded: false,
       errorMessage: null,
@@ -80,36 +89,18 @@ class PublishPage extends React.PureComponent {
       return;
     }
 
-    if (this.state.nameIsMine) {
+    let metadata = {};
-      // Pre-populate with existing metadata
-      var metadata = Object.assign({}, this.state.myClaimMetadata);
-      if (this.refs.file.getValue() !== "") {
-        delete metadata.sources;
-      }
-    } else {
-      var metadata = {};
-    }
 
-    for (let metaField of [
+    for (let metaField of ["title", "description", "thumbnail", "language"]) {
-      "title",
+      const value = this.state["meta_" + metaField];
-      "description",
+      if (value) {
-      "thumbnail",
-      "license",
-      "license_url",
-      "language",
-    ]) {
-      var value = this.refs["meta_" + metaField].getValue();
-      if (value !== "") {
         metadata[metaField] = value;
       }
     }
 
-    metadata.nsfw = parseInt(this.refs.meta_nsfw.getValue()) === 1;
+    metadata.license = this.getLicense();
-
+    metadata.licenseUrl = this.getLicenseUrl();
-    const licenseUrl = this.refs.meta_license_url.getValue();
+    metadata.nsfw = !!parseInt(this.state.meta_nsfw);
-    if (licenseUrl) {
-      metadata.license_url = licenseUrl;
-    }
 
     var doPublish = () => {
       var publishArgs = {
@@ -203,6 +194,8 @@ class PublishPage extends React.PureComponent {
   }
 
   myClaimInfo() {
+    const { name } = this.state;
+
     return Object.values(this.props.myClaims).find(
       claim => claim.name === name
     );
@@ -240,6 +233,7 @@ class PublishPage extends React.PureComponent {
     this.setState({
       rawName: rawName,
       name: name,
+      prefillDone: false,
       uri,
     });
 
@@ -254,6 +248,43 @@ class PublishPage extends React.PureComponent {
     });
   }
 
+  handlePrefillClicked() {
+    const {license, licenseUrl, title, thumbnail, description,
+           language, nsfw} = this.myClaimInfo().value.stream.metadata;
+
+    let newState = {
+      meta_title: title,
+      meta_thumbnail: thumbnail,
+      meta_description: description,
+      meta_language: language,
+      meta_nsfw: nsfw,
+    };
+
+    if (license == this._defaultCopyrightNotice) {
+      newState.licenseType = "copyright";
+      newState.copyrightNotice = this._defaultCopyrightNotice;
+    } else {
+      // If the license URL or description matches one of the drop-down options, use that
+      let licenseType = "other"; // Will be overridden if we find a match
+      for (let option of this._meta_license.getOptions()) {
+        if (
+          option.getAttribute("data-url") === licenseUrl ||
+          option.text === license
+        ) {
+          licenseType = option.value;
+        }
+      }
+
+      if (licenseType == "other") {
+        newState.otherLicenseDescription = license;
+        newState.otherLicenseUrl = licenseUrl;
+      }
+      newState.licenseType = licenseType;
+    }
+
+    this.setState(newState);
+  }
+
   handleBidChange(event) {
     this.setState({
       bid: event.target.value,
@@ -278,20 +309,21 @@ class PublishPage extends React.PureComponent {
     });
   }
 
-  handleLicenseChange(event) {
+  handleMetadataChange(event) {
-    var licenseType = event.target.options[
+    /**
-      event.target.selectedIndex
+     * This function is used for all metadata inputs that store the final value directly into state.
-    ].getAttribute("data-license-type");
+     * The only exceptions are inputs related to license description and license URL, which require
-    var newState = {
+     * more complex logic and the final value is determined at submit time.
-      copyrightChosen: licenseType == "copyright",
+     */
-      otherLicenseChosen: licenseType == "other",
+    this.setState({
-    };
+      ["meta_" + event.target.name]: event.target.value,
-
+    });
-    if (licenseType == "copyright") {
-      newState.copyrightNotice = __("All rights reserved.");
   }
 
-    this.setState(newState);
+  handleLicenseTypeChange(event) {
+    this.setState({
+      licenseType: event.target.value,
+    });
   }
 
   handleCopyrightNoticeChange(event) {
@@ -322,7 +354,7 @@ class PublishPage extends React.PureComponent {
 
   handleTOSChange(event) {
     this.setState({
-      TOSAgreed: event.target.checked,
+      tosAgree: event.target.checked,
     });
   }
 
@@ -366,16 +398,25 @@ class PublishPage extends React.PureComponent {
       );
   }
 
+  getLicense() {
+    switch (this.state.licenseType) {
+      case "copyright":
+        return this.state.copyrightNotice;
+      case "other":
+        return this.state.otherLicenseDescription;
+      default:
+        return this._meta_license.getSelectedElement().text;
+    }
+  }
+
   getLicenseUrl() {
-    if (!this.refs.meta_license) {
+    switch (this.state.licenseType) {
+      case "copyright":
         return "";
-    } else if (this.state.otherLicenseChosen) {
+      case "other":
         return this.state.otherLicenseUrl;
-    } else {
+      default:
-      return (
+        return this._meta_license.getSelectedElement().getAttribute("data-url");
-        this.refs.meta_license.getSelectedElement().getAttribute("data-url") ||
-        ""
-      );
     }
   }
 
@@ -398,7 +439,7 @@ class PublishPage extends React.PureComponent {
       this.props.resolvingUris.indexOf(this.state.uri) !== -1 &&
       this.claim() === undefined
     ) {
-      return <BusyMessage />;
+      return __("Checking...");
     } else if (!this.state.name) {
       return __("Select a URL for this publish.");
     } else if (!this.claim()) {
@@ -482,43 +523,55 @@ class PublishPage extends React.PureComponent {
                 }
               />
             </div>
-            {!this.state.hasFile
+            {!this.state.hasFile && !this.myClaimExists()
-              ? ""
+              ? null
               : <div>
                   <div className="card__content">
                     <FormRow
                       label={__("Title")}
                       type="text"
-                      ref="meta_title"
                       name="title"
-                      placeholder={__("Title")}
+                      value={this.state.meta_title}
+                      placeholder="Titular Title"
+                      onChange={event => {
+                        this.handleMetadataChange(event);
+                      }}
                     />
                   </div>
                   <div className="card__content">
                     <FormRow
                       type="text"
                       label={__("Thumbnail URL")}
-                      ref="meta_thumbnail"
                       name="thumbnail"
+                      value={this.state.meta_thumbnail}
                       placeholder="http://spee.ch/mylogo"
+                      onChange={event => {
+                        this.handleMetadataChange(event);
+                      }}
                     />
                   </div>
                   <div className="card__content">
                     <FormRow
                       label={__("Description")}
-                      type="textarea"
+                      type="SimpleMDE"
                       ref="meta_description"
                       name="description"
+                      value={this.state.meta_description}
                       placeholder={__("Description of your content")}
+                      onChange={event => {
+                        this.handleMetadataChange(event);
+                      }}
                     />
                   </div>
                   <div className="card__content">
                     <FormRow
                       label={__("Language")}
                       type="select"
-                      defaultValue="en"
+                      value={this.state.meta_language}
-                      ref="meta_language"
                       name="language"
+                      onChange={event => {
+                        this.handleMetadataChange(event);
+                      }}
                     >
                       <option value="en">{__("English")}</option>
                       <option value="zh">{__("Chinese")}</option>
@@ -533,9 +586,11 @@ class PublishPage extends React.PureComponent {
                     <FormRow
                       type="select"
                       label={__("Maturity")}
-                      defaultValue="en"
+                      value={this.state.meta_nsfw}
-                      ref="meta_nsfw"
                       name="nsfw"
+                      onChange={event => {
+                        this.handleMetadataChange(event);
+                      }}
                     >
                       {/* <option value=""></option> */}
                       <option value="0">{__("All Ages")}</option>
@@ -583,8 +638,7 @@ class PublishPage extends React.PureComponent {
                   placeholder="1.00"
                   min="0.01"
                   onChange={event => this.handleFeeAmountChange(event)}
-                />
+                />{" "}
-                {" "}
                 <FormField
                   type="select"
                   onChange={event => {
@@ -605,66 +659,71 @@ class PublishPage extends React.PureComponent {
               <FormRow
                 label="License"
                 type="select"
-                ref="meta_license"
+                value={this.state.licenseType}
-                name="license"
+                ref={row => {
+                  this._meta_license = row;
+                }}
                 onChange={event => {
-                  this.handleLicenseChange(event);
+                  this.handleLicenseTypeChange(event);
                 }}
               >
                 <option />
-                <option>{__("Public Domain")}</option>
+                <option value="publicDomain">{__("Public Domain")}</option>
-                <option data-url="https://creativecommons.org/licenses/by/4.0/legalcode">
+                <option
+                  value="cc-by"
+                  data-url="https://creativecommons.org/licenses/by/4.0/legalcode"
+                >
                   {__("Creative Commons Attribution 4.0 International")}
                 </option>
-                <option data-url="https://creativecommons.org/licenses/by-sa/4.0/legalcode">
+                <option
+                  value="cc-by-sa"
+                  data-url="https://creativecommons.org/licenses/by-sa/4.0/legalcode"
+                >
                   {__(
                     "Creative Commons Attribution-ShareAlike 4.0 International"
                   )}
                 </option>
-                <option data-url="https://creativecommons.org/licenses/by-nd/4.0/legalcode">
+                <option
+                  value="cc-by-nd"
+                  data-url="https://creativecommons.org/licenses/by-nd/4.0/legalcode"
+                >
                   {__(
                     "Creative Commons Attribution-NoDerivatives 4.0 International"
                   )}
                 </option>
-                <option data-url="https://creativecommons.org/licenses/by-nc/4.0/legalcode">
+                <option
+                  value="cc-by-nc"
+                  data-url="https://creativecommons.org/licenses/by-nc/4.0/legalcode"
+                >
                   {__(
                     "Creative Commons Attribution-NonCommercial 4.0 International"
                   )}
                 </option>
-                <option data-url="https://creativecommons.org/licenses/by-nc-sa/4.0/legalcode">
+                <option
+                  value="cc-by-nc-sa"
+                  data-url="https://creativecommons.org/licenses/by-nc-sa/4.0/legalcode"
+                >
                   {__(
                     "Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International"
                   )}
                 </option>
-                <option data-url="https://creativecommons.org/licenses/by-nc-nd/4.0/legalcode">
+                <option
+                  value="cc-by-nc-nd"
+                  data-url="https://creativecommons.org/licenses/by-nc-nd/4.0/legalcode"
+                >
                   {__(
                     "Creative Commons Attribution-NonCommercial-NoDerivatives 4.0 International"
                   )}
                 </option>
-                <option
+                <option value="copyright">
-                  data-license-type="copyright"
-                  {...(this.state.copyrightChosen
-                    ? { value: this.state.copyrightNotice }
-                    : {})}
-                >
                   {__("Copyrighted...")}
                 </option>
-                <option
+                <option value="other">
-                  data-license-type="other"
-                  {...(this.state.otherLicenseChosen
-                    ? { value: this.state.otherLicenseDescription }
-                    : {})}
-                >
                   {__("Other...")}
                 </option>
               </FormRow>
-              <FormField
+
-                type="hidden"
+              {this.state.licenseType == "copyright"
-                ref="meta_license_url"
-                name="license_url"
-                value={this.getLicenseUrl()}
-              />
-              {this.state.copyrightChosen
                 ? <FormRow
                     label={__("Copyright notice")}
                     type="text"
@@ -675,21 +734,25 @@ class PublishPage extends React.PureComponent {
                     }}
                   />
                 : null}
-              {this.state.otherLicenseChosen
+
+              {this.state.licenseType == "other"
                 ? <FormRow
                     label={__("License description")}
                     type="text"
                     name="other-license-description"
+                    value={this.state.otherLicenseDescription}
                     onChange={event => {
-                      this.handleOtherLicenseDescriptionChange();
+                      this.handleOtherLicenseDescriptionChange(event);
                     }}
                   />
                 : null}
-              {this.state.otherLicenseChosen
+
+              {this.state.licenseType == "other"
                 ? <FormRow
                     label={__("License URL")}
                     type="text"
                     name="other-license-url"
+                    value={this.state.otherLicenseUrl}
                     onChange={event => {
                       this.handleOtherLicenseUrlChange(event);
                     }}
@@ -730,6 +793,15 @@ class PublishPage extends React.PureComponent {
                 }}
                 helper={this.getNameBidHelpText()}
               />
+              {this.myClaimExists() && !this.state.prefillDone
+                ? <Notice>
+                    {__("You already have a claim with this name.")}{" "}
+                    <Link
+                      label={__("Use data from my existing claim")}
+                      onClick={() => this.handlePrefillClicked()}
+                    />
+                  </Notice>
+                : null}
             </div>
             {this.state.rawName
               ? <div className="card__content">
@@ -763,15 +835,11 @@ class PublishPage extends React.PureComponent {
                     <Link
                       href="https://www.lbry.io/termsofservice"
                       label={__("LBRY terms of service")}
-                      checked={this.state.TOSAgreed}
                     />
                   </span>
                 }
                 type="checkbox"
-                name="tos_agree"
+                checked={this.state.tosAgree}
-                ref={field => {
-                  this.refs.tos_agree = field;
-                }}
                 onChange={event => {
                   this.handleTOSChange(event);
                 }}

ui/package.json

@@ -29,8 +29,10 @@
     "rc-progress": "^2.0.6",
     "react": "^15.4.0",
     "react-dom": "^15.4.0",
+    "react-markdown": "^2.5.0",
     "react-modal": "^1.5.2",
     "react-redux": "^5.0.3",
+    "react-simplemde-editor": "^3.6.11",
     "redux": "^3.6.0",
     "redux-action-buffer": "^1.1.0",
     "redux-logger": "^3.0.1",
@@ -52,6 +54,8 @@
     "babel-preset-es2015": "^6.24.1",
     "babel-preset-react": "^6.24.1",
     "babel-preset-stage-2": "^6.18.0",
+    "electron-rebuild": "^1.5.11",
+    "css-loader": "^0.28.4",
     "eslint": "^3.10.2",
     "eslint-config-airbnb": "^13.0.0",
     "eslint-loader": "^1.6.1",
@@ -64,6 +68,7 @@
     "lint-staged": "^3.6.0",
     "node-loader": "^0.6.0",
     "prettier": "^1.4.2",
+    "style-loader": "^0.18.2",
     "webpack": "^2.6.1",
     "webpack-dev-server": "^2.4.4",
     "webpack-notifier": "^1.5.0",

ui/scss/component/_form-field.scss

@@ -117,6 +117,9 @@ input[type="text"].input-copyable {
     border: $width-input-border solid $color-form-border;
   }
 }
+.form-field--SimpleMDE {
+  display: block;
+}
 
 .form-field__label {
   &[for] {  cursor: pointer; }
@@ -164,3 +167,7 @@ input[type="text"].input-copyable {
 .form-field__helper {
   color: $color-help;
 }
+
+.form-field__input.form-field__input-SimpleMDE .CodeMirror-scroll {
+  height: auto;
+}