From 549ff33baf059df147cd84e221095562a997ce55 Mon Sep 17 00:00:00 2001
From: kobi <kobi.b@inner-active.com>
Date: Mon, 13 May 2019 14:03:40 +0300
Subject: [PATCH 1/4] wip

---
 src/mui/editModal.tsx           | 84 ++++++++++++---------------------
 src/mui/editModalButton.tsx     | 46 ++++++++++++++++++
 src/mui/editRemoteTextModal.tsx | 10 ++--
 3 files changed, 83 insertions(+), 57 deletions(-)
 create mode 100644 src/mui/editModalButton.tsx

diff --git a/src/mui/editModal.tsx b/src/mui/editModal.tsx
index 1856fa1..dbb3813 100644
--- a/src/mui/editModal.tsx
+++ b/src/mui/editModal.tsx
@@ -2,8 +2,6 @@ import Button from "@material-ui/core/Button"
 import Grid from "@material-ui/core/Grid"
 import Modal, {ModalProps} from "@material-ui/core/Modal"
 import Paper from "@material-ui/core/Paper"
-import {SvgIconProps} from "@material-ui/core/SvgIcon"
-import EditIcon from "@material-ui/icons/Edit"
 import * as React from "react"
 import {RemoteTextRecord} from "../core/remoteTextRecord"
 import {RemoteTextNode, RemoteTextValue} from "../core/remoteTextValue"
@@ -16,12 +14,10 @@ declare module "medium-editor" {
   }
 }
 
-export interface EditModalProps<T extends RemoteTextNode> {
+export interface EditModalProps<T extends RemoteTextNode> extends ModalProps {
   t: (document: T) => RemoteTextValue
   namespace?: string
-  modalProps?: Partial<ModalProps>
-  onModalOpen?: (open: boolean) => any
-  editIconProps?: SvgIconProps
+  onSave?: () => any
 }
 
 export interface EditModal<T extends RemoteTextNode> extends WithRemoteTextContext<T> {
@@ -48,19 +44,6 @@ export class EditModal<T extends RemoteTextNode> extends React.Component<EditMod
   el = document.createElement("div")
   mediumEditor: MediumEditor | null = null
 
-  state = {
-    modalOpen: false
-  }
-
-  changeModalOpen = (modalOpen: boolean) => {
-    this.setState({modalOpen}, () => {
-      const {onModalOpen} = this.props
-      if (onModalOpen != null) {
-        onModalOpen(this.state.modalOpen)
-      }
-    })
-  }
-
   componentDidMount() {
     document.body.appendChild(this.el)
   }
@@ -86,7 +69,10 @@ export class EditModal<T extends RemoteTextNode> extends React.Component<EditMod
             id: t(this.remoteTextStore.document).id
           })
         )
-        this.changeModalOpen(false)
+        const {onSave} = this.props
+        if (onSave != null) {
+          onSave()
+        }
       }
     }
   }
@@ -96,40 +82,32 @@ export class EditModal<T extends RemoteTextNode> extends React.Component<EditMod
   }
 
   render() {
-    const {t, modalProps, editIconProps, onModalOpen, namespace} = this.props
-    const {modalOpen} = this.state
-    const closeModal = () => this.changeModalOpen(false)
-
-    return <>
-      <EditIcon style={{cursor: "pointer"}} onClick={() => this.changeModalOpen(true)} {...editIconProps}/>
-
-      <Modal
-        disableAutoFocus
-        disableEnforceFocus
-        disableRestoreFocus
-        open={modalOpen}
-        onClose={closeModal}
-        {...modalProps}
-      >
-        <Grid container direction="column" wrap="nowrap" component={Paper} style={styles.modalBody}>
-          <Grid container direction="column" item style={{flex: 1}}>
-            <MediumEditor
-              style={{flex: 1}}
-              ref={this.mediumEditorRef}
-              text={t(this.remoteTextStore.document).html}
-            />
-          </Grid>
+    const {t, namespace, ...rest} = this.props
+
+    return <Modal
+      disableAutoFocus
+      disableEnforceFocus
+      disableRestoreFocus
+      {...rest}
+    >
+      <Grid container direction="column" wrap="nowrap" component={Paper} style={styles.modalBody}>
+        <Grid container direction="column" item style={{flex: 1}}>
+          <MediumEditor
+            style={{flex: 1}}
+            ref={this.mediumEditorRef}
+            text={t(this.remoteTextStore.document).html}
+          />
+        </Grid>
 
-          <Grid item container direction="row" style={{margin: "8px 4px"}} justify="flex-end">
-            <Button color="primary">
-              Cancel
-            </Button>
-            <Button onClick={this.handleSave} color="primary">
-              Save
-            </Button>
-          </Grid>
+        <Grid item container direction="row" style={{margin: "8px 4px"}} justify="flex-end">
+          <Button color="primary">
+            Cancel
+          </Button>
+          <Button onClick={this.handleSave} color="primary">
+            Save
+          </Button>
         </Grid>
-      </Modal>
-    </>
+      </Grid>
+    </Modal>
   }
 }
diff --git a/src/mui/editModalButton.tsx b/src/mui/editModalButton.tsx
new file mode 100644
index 0000000..64bf5ad
--- /dev/null
+++ b/src/mui/editModalButton.tsx
@@ -0,0 +1,46 @@
+import {SvgIconProps} from "@material-ui/core/SvgIcon"
+import EditIcon from "@material-ui/icons/Edit"
+import * as React from "react"
+import {RemoteTextNode, RemoteTextValue} from "../core/remoteTextValue"
+import {WithRemoteTextContext} from "../react/withRemoteText"
+import {EditModal, EditModalProps} from "./editModal"
+
+export interface EditModalButtonProps<T extends RemoteTextNode> {
+  t: (document: T) => RemoteTextValue
+  namespace?: string
+  editModalProps?: Partial<EditModalProps<T>>
+  onModalOpen?: (open: boolean) => any
+  editIconProps?: SvgIconProps
+}
+
+export interface EditModalButton<T extends RemoteTextNode> extends WithRemoteTextContext<T> {
+
+}
+
+export class EditModalButton<T extends RemoteTextNode> extends React.Component<EditModalButtonProps<T>> {
+  state = {
+    modalOpen: false
+  }
+
+  changeModalOpen = (modalOpen: boolean) => {
+    this.setState({modalOpen}, () => {
+      const {onModalOpen} = this.props
+      if (onModalOpen != null) {
+        onModalOpen(this.state.modalOpen)
+      }
+    })
+  }
+
+  render() {
+    const {t, editModalProps, editIconProps, onModalOpen, namespace} = this.props
+    const {modalOpen} = this.state
+    const closeModal = () => this.changeModalOpen(false)
+
+    return <>
+      <EditIcon style={{cursor: "pointer"}} onClick={() => this.changeModalOpen(true)} {...editIconProps}/>
+
+      <EditModal t={t} open={modalOpen} onClose={closeModal} onSave={closeModal}
+                 namespace={namespace} {...editModalProps}/>
+    </>
+  }
+}
diff --git a/src/mui/editRemoteTextModal.tsx b/src/mui/editRemoteTextModal.tsx
index 3ec848c..e91d90c 100644
--- a/src/mui/editRemoteTextModal.tsx
+++ b/src/mui/editRemoteTextModal.tsx
@@ -5,14 +5,15 @@ import * as React from "react"
 import {RemoteTextNode, RemoteTextValue} from "../core/remoteTextValue"
 import {RemoteText} from "../react/remoteText"
 import {withRemoteText, WithRemoteTextContext} from "../react/withRemoteText"
-import {EditModal} from "./editModal"
+import {EditModal, EditModalProps} from "./editModal"
+import {EditModalButton} from "./editModalButton"
 
 export interface EditRemoteTextModalProps<T extends RemoteTextNode> extends Partial<GridProps> {
   t: (document: T) => RemoteTextValue
   namespace?: string
   children: (value: RemoteTextValue) => React.ReactElement<any>
   editIconProps?: SvgIconProps
-  modalProps?: Partial<ModalProps>
+  editModalProps?: Partial<EditModalProps<T>>
   onModalOpen?: (open: boolean) => any
 }
 
@@ -29,7 +30,7 @@ export class EditRemoteTextModal<T extends RemoteTextNode> extends React.Compone
   changeShowEditIcon = (showEditIcon: boolean) => this.setState({showEditIcon})
 
   render() {
-    const {t, children, modalProps, onModalOpen, namespace, editIconProps, ...rest} = this.props
+    const {t, children, editModalProps, onModalOpen, namespace, editIconProps, ...rest} = this.props
     const {showEditIcon} = this.state
     return <>
       <Grid container direction="row" wrap="nowrap" alignItems="center"
@@ -39,7 +40,8 @@ export class EditRemoteTextModal<T extends RemoteTextNode> extends React.Compone
           <RemoteText t={t}>{children}</RemoteText>
         </Grid>
         <Grid item>
-          {showEditIcon && <EditModal t={t} modalProps={modalProps} onModalOpen={onModalOpen} namespace={namespace}/>}
+          {showEditIcon &&
+          <EditModalButton t={t} editModalProps={editModalProps} onModalOpen={onModalOpen} namespace={namespace}/>}
         </Grid>
       </Grid>
     </>

From ce58adb86190a089ca48ff26b0e25aac708ba553 Mon Sep 17 00:00:00 2001
From: kobi <kobi.b@inner-active.com>
Date: Mon, 13 May 2019 16:13:34 +0300
Subject: [PATCH 2/4] wip

---
 src/mui/editModal.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mui/editModal.tsx b/src/mui/editModal.tsx
index dbb3813..a329167 100644
--- a/src/mui/editModal.tsx
+++ b/src/mui/editModal.tsx
@@ -82,7 +82,7 @@ export class EditModal<T extends RemoteTextNode> extends React.Component<EditMod
   }
 
   render() {
-    const {t, namespace, ...rest} = this.props
+    const {t, namespace, onSave, ...rest} = this.props
 
     return <Modal
       disableAutoFocus

From e0f46578663649b4a667ed53a960f45de9fe0d98 Mon Sep 17 00:00:00 2001
From: kobi <kobi.b@inner-active.com>
Date: Wed, 15 May 2019 21:05:56 +0300
Subject: [PATCH 3/4] wip

---
 src/mongo/remoteTextMongo.ts | 11 ++++++++++-
 src/react/mediumEditor.tsx   |  3 ++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/mongo/remoteTextMongo.ts b/src/mongo/remoteTextMongo.ts
index c274ae3..68019e8 100644
--- a/src/mongo/remoteTextMongo.ts
+++ b/src/mongo/remoteTextMongo.ts
@@ -6,6 +6,14 @@ import {RemoteTextDocument} from "../core/remoteTextValue"
 
 const allowedTags = sanitizeHtml.defaults.allowedTags.filter(e => e !== "iframe")
 
+const allowedAttributes = {
+  a: ["href", "name", "target", "style"],
+  img: ["src"],
+  p: ["style"],
+  span: ["style"],
+  div: ["style"]
+}
+
 export interface RemoteTextSchema {
   namespace: string
   lang: string
@@ -47,7 +55,8 @@ export class RemoteTextMongo {
       $set: {
         [`document.${record.id}`]: {
           html: sanitizeHtml(record.html, {
-            allowedTags
+            allowedTags,
+            allowedAttributes
           })
         }
       }
diff --git a/src/react/mediumEditor.tsx b/src/react/mediumEditor.tsx
index 0a3ba3b..2e1bced 100644
--- a/src/react/mediumEditor.tsx
+++ b/src/react/mediumEditor.tsx
@@ -15,7 +15,8 @@ export class MediumEditor extends React.Component<MediumEditorProps> {
     this.medium = new MediumEditorBase(dom, {
       paste: {
         forcePlainText: false
-      }
+      },
+      targetBlank: true
     })
     this.medium.subscribe("editableInput", () => {
       const {onTextChange} = this.props

From 0e317216be1805600fbe774c92ecc3704f553496 Mon Sep 17 00:00:00 2001
From: kobi <kobi.b@inner-active.com>
Date: Wed, 15 May 2019 22:53:00 +0300
Subject: [PATCH 4/4] wip

---
 src/mongo/remoteTextMongo.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mongo/remoteTextMongo.ts b/src/mongo/remoteTextMongo.ts
index 68019e8..e428e98 100644
--- a/src/mongo/remoteTextMongo.ts
+++ b/src/mongo/remoteTextMongo.ts
@@ -4,7 +4,7 @@ import {Option} from "tsla-util/lib/option"
 import {RemoteTextRecord} from "../core/remoteTextRecord"
 import {RemoteTextDocument} from "../core/remoteTextValue"
 
-const allowedTags = sanitizeHtml.defaults.allowedTags.filter(e => e !== "iframe")
+const allowedTags = sanitizeHtml.defaults.allowedTags.filter(e => e !== "iframe").concat("span")
 
 const allowedAttributes = {
   a: ["href", "name", "target", "style"],