From 7a6776c61b5af4eae7393565df7665d4cbfdb8a0 Mon Sep 17 00:00:00 2001
From: muerwre <gotham48@gmail.com>
Date: Tue, 27 Nov 2018 15:05:49 +0700
Subject: [PATCH] save: save reset appearance: save button reacts to changes

---
 src/components/panels/EditorPanel.jsx |  6 +++---
 src/constants/tips.js                 |  5 +++--
 src/modules/Editor.js                 | 16 ++++++++++++----
 src/redux/user/actions.js             |  2 +-
 src/redux/user/constants.js           |  1 +
 src/redux/user/reducer.js             | 16 +++++++++++-----
 src/redux/user/sagas.js               |  2 +-
 src/styles/panel.less                 |  6 ++++--
 8 files changed, 36 insertions(+), 18 deletions(-)

diff --git a/src/components/panels/EditorPanel.jsx b/src/components/panels/EditorPanel.jsx
index b2a2b49..5c02b2f 100644
--- a/src/components/panels/EditorPanel.jsx
+++ b/src/components/panels/EditorPanel.jsx
@@ -54,7 +54,7 @@ class Component extends React.PureComponent<Props, void> {
 
   render() {
     const {
-      mode, routerPoints, distance, estimated, activeSticker, logo, user, editing, title, address,
+      mode, distance, estimated, changed, logo, editing,
     } = this.props;
 
     return (
@@ -130,14 +130,14 @@ class Component extends React.PureComponent<Props, void> {
 
           <div className="control-bar">
             <button
-              className="highlighted  cancel"
+              className="highlighted cancel"
               onClick={this.props.stopEditing}
             >
               <Icon icon="icon-cancel-1" />
             </button>
 
             <button
-              className="primary"
+              className={classnames({ primary: changed, disabled: !changed })}
               onClick={this.startSaveMode}
             >
               <span>СХОРОНИТЬ</span>
diff --git a/src/constants/tips.js b/src/constants/tips.js
index 5fc206a..aeb3caf 100644
--- a/src/constants/tips.js
+++ b/src/constants/tips.js
@@ -1,6 +1,7 @@
 export const TIPS = {
   SAVE_INFO: 'Никто, кроме вас не сможет изменить маршрут - только создать его копию и сохранить по другому адресу',
-  SAVE_TIMED_OUT: 'Сервер не ответил на запрос, попробуйте позже',
+  SAVE_TIMED_OUT: 'Сервер не ответил на запрос, попробуйте еще раз',
   SAVE_EMPTY: 'Этот маршрут пуст, нарисуйте что-нибудь для начала',
-  SAVE_SUCCESS: 'Маршрут сохранен - он будет доступен по указанному адресу'
+  SAVE_SUCCESS: 'Отлично! Ваш маршрут сохранён. Поделитесь ссылкой с друзьями, приятной покатушки!',
+  SAVE_OVERWRITE: 'У вас уже есть маршрут с таким именем. Можно перезаписать его или переименовать?'
 };
diff --git a/src/modules/Editor.js b/src/modules/Editor.js
index 683ee2e..ad27e12 100644
--- a/src/modules/Editor.js
+++ b/src/modules/Editor.js
@@ -10,13 +10,13 @@ import { parseStickerAngle, parseStickerStyle } from '$utils/import';
 import { getUrlData, pushPath } from '$utils/history';
 import { store } from '$redux/store';
 import {
+  resetSaveDialog,
   setActiveSticker, setAddress,
   setChanged,
   setDistance,
-  setEditing,
-  setLogo, setMode,
+  setMode,
   setRouterPoints,
-  setTitle
+  setTitle,
 } from '$redux/user/actions';
 
 export class Editor {
@@ -67,6 +67,9 @@ export class Editor {
       },
       [MODES.LOGO]: {
         toggle: this.clearMode,
+      },
+      [MODES.SAVE]: {
+        stop: this.resetSaveDialog,
       }
     };
 
@@ -98,16 +101,21 @@ export class Editor {
   getDistance = () => store.getState().user.distance;
 
   setMode = value => store.dispatch(setMode(value));
-  setChanged = value => store.dispatch(setChanged(value));
   setRouterPoints = value => store.dispatch(setRouterPoints(value));
   setActiveSticker = value => store.dispatch(setActiveSticker(value));
   setTitle = value => store.dispatch(setTitle(value));
   setAddress = value => store.dispatch(setAddress(value));
 
+  resetSaveDialog = () => store.dispatch(resetSaveDialog());
+
   setDistance = value => {
     if (this.getDistance() !== value) store.dispatch(setDistance(value));
   };
 
+  setChanged = value => {
+    if (this.getChanged() !== value) store.dispatch(setChanged(value));
+  };
+
   clearMode = () => this.setMode(MODES.NONE);
   clearChanged = () => store.dispatch(setChanged(false));
 
diff --git a/src/redux/user/actions.js b/src/redux/user/actions.js
index 2edd440..4f3af93 100644
--- a/src/redux/user/actions.js
+++ b/src/redux/user/actions.js
@@ -26,7 +26,7 @@ export const clearAll = () => ({ type: ACTIONS.CLEAR_ALL });
 export const clearCancel = () => ({ type: ACTIONS.CLEAR_CANCEL });
 
 export const sendSaveRequest = payload => ({ type: ACTIONS.SEND_SAVE_REQUEST, ...payload });
-export const cancelSaveRequest = () => ({ type: ACTIONS.CANCEL_SAVE_REQUEST });
+export const resetSaveDialog = () => ({ type: ACTIONS.RESET_SAVE_DIALOG });
 
 export const setSaveSuccess = payload => ({ type: ACTIONS.SET_SAVE_SUCCESS, ...payload });
 export const setSaveError = save_error => ({ type: ACTIONS.SET_SAVE_ERROR, save_error });
diff --git a/src/redux/user/constants.js b/src/redux/user/constants.js
index f61d967..45ac0ba 100644
--- a/src/redux/user/constants.js
+++ b/src/redux/user/constants.js
@@ -25,6 +25,7 @@ export const ACTIONS = {
 
   SEND_SAVE_REQUEST: 'SEND_SAVE_REQUEST',
   CANCEL_SAVE_REQUEST: 'CANCEL_SAVE_REQUEST',
+  RESET_SAVE_DIALOG: 'RESET_SAVE_DIALOG',
 
   SET_SAVE_SUCCESS: 'SET_SAVE_SUCCESS',
   SET_SAVE_ERROR: 'SET_SAVE_ERROR',
diff --git a/src/redux/user/reducer.js b/src/redux/user/reducer.js
index e6510cb..a084ca2 100644
--- a/src/redux/user/reducer.js
+++ b/src/redux/user/reducer.js
@@ -3,6 +3,7 @@ import { ACTIONS } from '$redux/user/constants';
 import { DEFAULT_USER } from '$constants/auth';
 import { MODES } from '$constants/modes';
 import { DEFAULT_LOGO } from '$constants/logos';
+import { TIPS } from '$constants/tips';
 
 const getEstimated = distance => {
   const time = (distance && (distance / 15)) || 0;
@@ -18,9 +19,7 @@ const setUser = (state, { user }) => ({
 });
 
 const setEditing = (state, { editing }) => ({ ...state, editing });
-const setChanged = (state, { changed }) => ({
-  ...state, changed, ...state, save_overwriting: false, save_finished: false, save_processing: false, save_error: '',
-});
+const setChanged = (state, { changed }) => ({ ...state, changed });
 const setMode = (state, { mode }) => ({ ...state, mode });
 const setDistance = (state, { distance }) => ({
   ...state,
@@ -39,13 +38,19 @@ const sendSaveRequest = state => ({ ...state, save_processing: true, });
 const setSaveError = (state, { save_error }) => ({
   ...state, save_error, save_finished: false, save_processing: false
 });
+
 const setSaveOverwrite = state => ({
-  ...state, save_overwriting: true, save_finished: false, save_processing: false
+  ...state, save_overwriting: true, save_finished: false, save_processing: false, save_error: TIPS.SAVE_OVERWRITE,
 });
+
 const setSaveSuccess = (state, { save_error }) => ({
   ...state, save_overwriting: false, save_finished: true, save_processing: false, save_error
 });
 
+const resetSaveDialog = state => ({
+  ...state, save_overwriting: false, save_finished: false, save_processing: false, save_error: '',
+});
+
 const HANDLERS = {
   [ACTIONS.SET_USER]: setUser,
   [ACTIONS.SET_EDITING]: setEditing,
@@ -62,6 +67,7 @@ const HANDLERS = {
   [ACTIONS.SET_SAVE_OVERWRITE]: setSaveOverwrite,
   [ACTIONS.SET_SAVE_SUCCESS]: setSaveSuccess,
   [ACTIONS.SEND_SAVE_REQUEST]: sendSaveRequest,
+  [ACTIONS.RESET_SAVE_DIALOG]: resetSaveDialog,
 };
 
 export const INITIAL_STATE = {
@@ -73,7 +79,7 @@ export const INITIAL_STATE = {
   distance: 0,
   estimated: 0,
   activeSticker: null,
-  title: 0,
+  title: '',
   address: '',
   changed: false,
 
diff --git a/src/redux/user/sagas.js b/src/redux/user/sagas.js
index 9910c79..d7b09e2 100644
--- a/src/redux/user/sagas.js
+++ b/src/redux/user/sagas.js
@@ -195,7 +195,7 @@ function* sendSaveRequestSaga({ title, address, force }) {
       id, token, route, stickers, title, force, address
     }),
     timeout: delay(10000),
-    cancel: take(ACTIONS.CANCEL_SAVE_REQUEST),
+    cancel: take(ACTIONS.RESET_SAVE_DIALOG),
   });
 
   if (cancel) return yield put(setMode(MODES.NONE));
diff --git a/src/styles/panel.less b/src/styles/panel.less
index cf1c2a9..dae14fb 100644
--- a/src/styles/panel.less
+++ b/src/styles/panel.less
@@ -97,17 +97,19 @@
     }
 
     &.disabled {
+      color: #999999;
+
       svg {
         fill: #999999;
       }
     }
 
     &.highlighted {
-      background: #555555;
+      background: rgba(255, 255, 255, 0.1);
     }
 
     &.cancel {
-      background: linear-gradient(270deg, #0f5871, #444444 60%);
+      // background: linear-gradient(270deg, #0f5871, #444444 60%);
     }
 
     &.primary {