Support markdown makeup in claim description #242

Closed
longle255 wants to merge 11 commits from claim-desc-markdown-support into master
6 changed files with 54 additions and 21 deletions
Showing only changes of commit bcfda3ffa1 - Show all commits

2
.gitignore vendored
View file

@ -17,3 +17,5 @@ dist
build/daemon.zip build/daemon.zip
.vimrc .vimrc
ui/yarn.lock

View file

@ -59,5 +59,9 @@
"electron": "^1.4.15", "electron": "^1.4.15",
"electron-builder": "^11.7.0", "electron-builder": "^11.7.0",
"electron-debug": "^1.1.0" "electron-debug": "^1.1.0"
},
"dependencies": {
"react-simplemde-editor": "^3.6.11",
"style-loader": "^0.18.2"
} }
} }

View file

@ -1,6 +1,8 @@
import React from "react"; import React from "react";
import FileSelector from "./file-selector.js"; import FileSelector from "./file-selector.js";
import { Icon } from "./common.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, var formFieldCounter = 0,
formFieldFileSelectorTypes = ["file", "directory"], formFieldFileSelectorTypes = ["file", "directory"],
@ -38,6 +40,9 @@ export class FormField extends React.PureComponent {
} else if (this.props.type == "text-number") { } else if (this.props.type == "text-number") {
this._element = "input"; this._element = "input";
this._type = "text"; this._type = "text";
} else if (this.props.type == "SimpleMDE") {
this._element = "SimpleMDE";
this._type = "textarea";
} else if (formFieldFileSelectorTypes.includes(this.props.type)) { } else if (formFieldFileSelectorTypes.includes(this.props.type)) {
this._element = "input"; this._element = "input";
this._type = "hidden"; this._type = "hidden";
@ -106,26 +111,42 @@ export class FormField extends React.PureComponent {
delete otherProps.className; delete otherProps.className;
delete otherProps.postfix; delete otherProps.postfix;
kauffj commented 2017-06-16 23:26:32 +02:00 (Migrated from github.com)
Review

I'm decently confident this is possible, maybe with this._element = <SimpleMDE> or this._element = SimpleMDE? Worth looking into. If it's not, this can still be cleaned up by compacting the shared properties to one declaration and passing them in with an extract operator.

I'm decently confident this is possible, maybe with `this._element = <SimpleMDE>` or `this._element = SimpleMDE`? Worth looking into. If it's not, this can still be cleaned up by compacting the shared properties to one declaration and passing them in with an extract operator.
delete otherProps.prefix; delete otherProps.prefix;
const element = this._element === "SimpleMDE"
const element = ( ? <SimpleMDE
<this._element id={elementId}
id={elementId} type={this._type}
type={this._type} name={this.props.name}
name={this.props.name} ref="field"
ref="field" placeholder={this.props.placeholder}
placeholder={this.props.placeholder} className={
className={ "form-field__input form-field__input-" +
"form-field__input form-field__input-" + this.props.type +
this.props.type + " " +
" " + (this.props.className || "") +
(this.props.className || "") + (isError ? "form-field__input--error" : "")
(isError ? "form-field__input--error" : "") }
} name={this.props.name}
{...otherProps} {...otherProps}
> >
{this.props.children} {this.props.children}
</this._element> </SimpleMDE>
); : <this._element // can't be applied to SimpleMDE idk why
id={elementId}
type={this._type}
name={this.props.name}
ref="field"
placeholder={this.props.placeholder}
className={
"form-field__input form-field__input-" +
this.props.type +
" " +
(this.props.className || "") +
(isError ? "form-field__input--error" : "")
}
{...otherProps}
>
{this.props.children}
</this._element>;
return ( return (
<div className={"form-field form-field--" + this.props.type}> <div className={"form-field form-field--" + this.props.type}>

View file

@ -520,7 +520,7 @@ class PublishPage extends React.PureComponent {
<div className="card__content"> <div className="card__content">
<FormRow <FormRow
label={__("Description")} label={__("Description")}
type="textarea" type="SimpleMDE"
ref="meta_description" ref="meta_description"
name="description" name="description"
placeholder={__("Description of your content")} placeholder={__("Description of your content")}

View file

@ -54,6 +54,7 @@
"babel-preset-es2015": "^6.18.0", "babel-preset-es2015": "^6.18.0",
"babel-preset-react": "^6.16.0", "babel-preset-react": "^6.16.0",
"babel-preset-stage-2": "^6.18.0", "babel-preset-stage-2": "^6.18.0",
"css-loader": "^0.28.4",
"eslint": "^3.10.2", "eslint": "^3.10.2",
"eslint-config-airbnb": "^13.0.0", "eslint-config-airbnb": "^13.0.0",
"eslint-loader": "^1.6.1", "eslint-loader": "^1.6.1",
@ -65,6 +66,7 @@
"lint-staged": "^3.6.0", "lint-staged": "^3.6.0",
"node-sass": "^3.13.0", "node-sass": "^3.13.0",
"prettier": "^1.4.2", "prettier": "^1.4.2",
"style-loader": "^0.18.2",
"webpack": "^2.6.1", "webpack": "^2.6.1",
"webpack-dev-server": "^2.4.4", "webpack-dev-server": "^2.4.4",
"webpack-notifier": "^1.5.0", "webpack-notifier": "^1.5.0",

View file

@ -163,4 +163,8 @@ input[type="text"].input-copyable {
} }
.form-field__helper { .form-field__helper {
color: $color-help; color: $color-help;
}
.CodeMirror-scroll {
height: auto !important;
} }