From 1ceb1b4014bc16f2cee8d44495b0bf68948b8236 Mon Sep 17 00:00:00 2001
From: okbel <curciobelen@gmail.com>
Date: Tue, 10 Apr 2018 11:46:32 -0300
Subject: [PATCH] Save, and edit changes

---
 .../routes/Configure/components/Configure.js  | 13 +++-
 .../components/OrganizationSettings.css       | 27 +++++---
 .../components/OrganizationSettings.js        | 63 ++++++++++++++-----
 .../routes/Configure/containers/Configure.js  |  5 +-
 locales/en.yml                                |  2 +
 locales/es.yml                                |  2 +
 6 files changed, 88 insertions(+), 24 deletions(-)

diff --git a/client/coral-admin/src/routes/Configure/components/Configure.js b/client/coral-admin/src/routes/Configure/components/Configure.js
index 69ab4f06c..bcd48c21c 100644
--- a/client/coral-admin/src/routes/Configure/components/Configure.js
+++ b/client/coral-admin/src/routes/Configure/components/Configure.js
@@ -8,7 +8,14 @@ import SaveChangesDialog from './SaveChangesDialog';
 
 class Configure extends React.Component {
   render() {
-    const { canSave, currentUser, root, savePending, settings } = this.props;
+    const {
+      canSave,
+      currentUser,
+      root,
+      savePending,
+      settings,
+      clearPending,
+    } = this.props;
 
     if (!can(currentUser, 'UPDATE_CONFIG')) {
       return <p>{t('configure.access_message')}</p>;
@@ -17,6 +24,9 @@ class Configure extends React.Component {
     const passProps = {
       root,
       settings,
+      savePending,
+      clearPending,
+      canSave,
     };
 
     return (
@@ -84,6 +94,7 @@ Configure.propTypes = {
   children: PropTypes.node.isRequired,
   saveDialog: PropTypes.bool,
   hideSaveDialog: PropTypes.func.isRequired,
+  clearPending: PropTypes.func.isRequired,
 };
 
 export default Configure;
diff --git a/client/coral-admin/src/routes/Configure/components/OrganizationSettings.css b/client/coral-admin/src/routes/Configure/components/OrganizationSettings.css
index 06ada55c4..d28fced83 100644
--- a/client/coral-admin/src/routes/Configure/components/OrganizationSettings.css
+++ b/client/coral-admin/src/routes/Configure/components/OrganizationSettings.css
@@ -16,7 +16,7 @@
 }
 
 .detailValue {
-  padding: 6px 10px;
+  padding: 6px 0;
   border: solid 1px transparent;
   display: block;
   font-size: 1em;
@@ -24,6 +24,8 @@
 }
 
 .editable {
+  padding-left: 10px;
+  padding-right: 10px;
   border-color: grey;
 }
 
@@ -32,20 +34,31 @@
   list-style: none;
 }
 
-.editButton {
-  position: absolute;
-  right: 10px;
-  top: 10px;
+.button, .button:disabled {
+  background: white;
+  border: solid 1px grey;
 }
 
 .actionBox {
   position: absolute;
-  right: 10px;
-  top: 10px; 
+  right: 20px;
+  top: 20px;
   text-align: center;
+  width: 100px;
 }
 
 .cancelButton {
   padding: 10px;
   display: block;
+  color: #4f5c67;
+  font-weight: 500;
+  
+  &:hover {
+    cursor: pointer;
+  }
+}
+
+.changedSave {
+  background-color: #00796B;
+  color: white;
 }
\ No newline at end of file
diff --git a/client/coral-admin/src/routes/Configure/components/OrganizationSettings.js b/client/coral-admin/src/routes/Configure/components/OrganizationSettings.js
index 18fe653e1..06c91926c 100644
--- a/client/coral-admin/src/routes/Configure/components/OrganizationSettings.js
+++ b/client/coral-admin/src/routes/Configure/components/OrganizationSettings.js
@@ -17,6 +17,12 @@ class OrganizationSettings extends React.Component {
     }));
   };
 
+  disableEditing = () => {
+    this.setState(() => ({
+      editing: false,
+    }));
+  };
+
   updateName = event => {
     const updater = { organizationName: { $set: event.target.value } };
     this.props.updatePending({ updater });
@@ -27,32 +33,56 @@ class OrganizationSettings extends React.Component {
     this.props.updatePending({ updater });
   };
 
+  cancelEditing = () => {
+    this.disableEditing();
+    this.props.clearPending();
+  };
+
+  save = async () => {
+    await this.props.savePending();
+    this.disableEditing();
+  };
+
   render() {
-    const { settings, slotPassthrough } = this.props;
+    const { settings, slotPassthrough, canSave } = this.props;
     return (
       <ConfigurePage title={t('configure.organization_information')}>
         <p>{t('configure.organization_info_copy')}</p>
         <p>{t('configure.organization_info_copy_2')}</p>
         <ConfigureCard>
           {!this.state.editing ? (
-            <Button
-              className={styles.editButton}
-              icon="settings"
-              onClick={this.toggleEditing}
-            >
-              Edit Info
-            </Button>
+            <div className={styles.actionBox}>
+              <Button
+                className={styles.button}
+                icon="settings"
+                onClick={this.toggleEditing}
+                full
+              >
+                {t('configure.edit_info')}
+              </Button>
+            </div>
           ) : (
             <div className={styles.actionBox}>
-              <Button icon="done" onClick={this.toggleEditing}>
-                Save
-              </Button>
-              <a className={styles.cancelButton}>Cancel</a>
+              {canSave ? (
+                <Button
+                  raised
+                  onClick={this.save}
+                  className={styles.changedSave}
+                  icon="check"
+                  full
+                >
+                  {t('configure.save')}
+                </Button>
+              ) : (
+                <Button className={styles.button} disabled icon="check" full>
+                  {t('configure.save')}
+                </Button>
+              )}
+              <a className={styles.cancelButton} onClick={this.cancelEditing}>
+                {t('cancel')}
+              </a>
             </div>
           )}
-
-          {console.log(this.props.slotPassthrough)}
-
           <ul className={styles.detailList}>
             <li className={styles.detailItem}>
               <label
@@ -99,10 +129,13 @@ class OrganizationSettings extends React.Component {
 }
 
 OrganizationSettings.propTypes = {
+  savePending: PropTypes.func.isRequired,
+  clearPending: PropTypes.func.isRequired,
   updatePending: PropTypes.func.isRequired,
   errors: PropTypes.object.isRequired,
   settings: PropTypes.object.isRequired,
   slotPassthrough: PropTypes.object.isRequired,
+  canSave: PropTypes.bool.isRequired,
 };
 
 export default OrganizationSettings;
diff --git a/client/coral-admin/src/routes/Configure/containers/Configure.js b/client/coral-admin/src/routes/Configure/containers/Configure.js
index 1ee521d69..7f0951154 100644
--- a/client/coral-admin/src/routes/Configure/containers/Configure.js
+++ b/client/coral-admin/src/routes/Configure/containers/Configure.js
@@ -84,18 +84,21 @@ class ConfigureContainer extends React.Component {
       return <Spinner />;
     }
 
+    const activeSection = this.props.routes[3].path;
+
     return (
       <Configure
         saveChanges={this.saveChanges}
         discardChanges={this.discardChanges}
         saveDialog={this.props.saveDialog}
-        activeSection={this.props.routes[3].path}
+        activeSection={activeSection}
         hideSaveDialog={this.props.hideSaveDialog}
         canSave={this.props.canSave}
         currentUser={this.props.currentUser}
         root={this.props.root}
         settings={this.props.mergedSettings}
         handleSectionChange={this.handleSectionChange}
+        clearPending={this.props.clearPending}
         savePending={this.savePending}
       >
         {this.props.children}
diff --git a/locales/en.yml b/locales/en.yml
index 2da3b3e4b..2f06eef5b 100644
--- a/locales/en.yml
+++ b/locales/en.yml
@@ -124,6 +124,7 @@ en:
     description: "As an admin, you can customize the settings for the comment stream for this story:"
     domain_list_text: "Enter the domains you would like to permit for Talk e.g. your local staging and production environments (ex. localhost:3000 staging.domain.com domain.com)."
     domain_list_title: "Permitted Domains"
+    edit_info: "Edit Info"
     edit_comment_timeframe_heading: "Edit Comment Timeframe"
     edit_comment_timeframe_text_pre: "Commenters will have"
     edit_comment_timeframe_text_post: "seconds to edit their comments."
@@ -149,6 +150,7 @@ en:
     open_stream_configuration: "This comment stream is currently open. By closing this comment stream no new comments may be submitted and all previous comments will still be displayed."
     require_email_verification: "Require Email Verification"
     require_email_verification_text: "New Users must verify their email before commenting"
+    save: Save
     save_changes: "Save Changes"
     shortcuts: Shortcuts
     sign_out: "Sign Out"
diff --git a/locales/es.yml b/locales/es.yml
index f6d587560..f68f51d0e 100644
--- a/locales/es.yml
+++ b/locales/es.yml
@@ -123,6 +123,7 @@ es:
     description: "Como Administrador/a puedes modificar la configuración de los comentarios en este artículo"
     domain_list_text: "Agrega dominios permitidos a Talk, por ejemplo tu localhost, staging y ambientes de producción (ej. localhost:3000, staging.domain.com, domain.com)."
     domain_list_title: "Dominios Permitidos"
+    edit_info: "Editar Información"
     edit_comment_timeframe_heading: "Periodo de Tiempo para Edición del Comentario"
     edit_comment_timeframe_text_pre: "Los comentaristas tendrán"
     edit_comment_timeframe_text_post: "segundos para editar sus comentarios."
@@ -148,6 +149,7 @@ es:
     open_stream_configuration: "Este hilo de comentarios está abierto. Al cerrarlo no se podrán publicar nuevos comentarios pero todos los comentarios anteriores aún serán mostrados."
     require_email_verification: "Necesita confirmación su correo"
     require_email_verification_text: "Nuevos usuarios deben confirmar sus correos antes de comentar"
+    save: Guardar
     save_changes: "Guardar Cambios"
     shortcuts: Atajos
     sign_out: "Desconectar"