From 4e0e11a6116f0d7a4d16d062962705a718ac6ef8 Mon Sep 17 00:00:00 2001
From: Rokt33r <fluke8259@gmail.com>
Date: Mon, 4 Jan 2016 19:38:51 +0900
Subject: [PATCH] add animation

---
 browser/main/HomePage/ArticleDetail/index.js | 14 +++++---
 browser/main/HomePage/ArticleNavigator.js    | 12 +++----
 browser/styles/main/ArticleDetail.styl       | 34 ++++++++++++++++++--
 browser/styles/main/ArticleNavigator.styl    |  8 ++++-
 4 files changed, 53 insertions(+), 15 deletions(-)

diff --git a/browser/main/HomePage/ArticleDetail/index.js b/browser/main/HomePage/ArticleDetail/index.js
index 7ab3019a..5fdc7814 100644
--- a/browser/main/HomePage/ArticleDetail/index.js
+++ b/browser/main/HomePage/ArticleDetail/index.js
@@ -34,7 +34,6 @@ othersMenu.append(new MenuItem({
   }
 }))
 
-const OSX = global.process.platform === 'darwin'
 const BRAND_COLOR = '#18AF90'
 
 const editDeleteTutorialElement = (
@@ -312,14 +311,21 @@ export default class ArticleDetail extends React.Component {
             />
 
             <div className='ArticleDetail-info-control'>
+              <div className={'ArticleDetail-info-control-save' + (!isUnsaved ? ' hide' : '')}>
+                <button
+                  onClick={e => this.handleSaveButtonClick(e)}
+                  className='ArticleDetail-info-control-save-button'
+                  disabled={!isUnsaved}
+                  >
+                  <i className='fa fa-fw fa-save'/>&nbsp;Save
+                </button>
+              </div>
+
               <ShareButton
                 article={activeArticle}
                 user={user}
                 />
 
-              <button onClick={e => this.handleSaveButtonClick(e)}>
-                <i className='fa fa-fw fa-save'/><span className='tooltip'>Save ({OSX ? '⌘ + s' : '^ + s'})</span>
-              </button>
               <button onClick={e => this.handleOthersButtonClick(e)}>
                 <i className='fa fa-fw fa-angle-down'/>
               </button>
diff --git a/browser/main/HomePage/ArticleNavigator.js b/browser/main/HomePage/ArticleNavigator.js
index 36128fae..e248477b 100644
--- a/browser/main/HomePage/ArticleNavigator.js
+++ b/browser/main/HomePage/ArticleNavigator.js
@@ -144,6 +144,7 @@ export default class ArticleNavigator extends React.Component {
         </div>
       )
     }).filter(modifiedArticle => modifiedArticle).sort((a, b) => a.updatedAt - b.updatedAt)
+    let hasModified = modifiedElements.length > 0
 
     let folderElememts = folders.map((folder, index) => {
       let isActive = findWhere(targetFolders, {key: folder.key})
@@ -170,22 +171,17 @@ export default class ArticleNavigator extends React.Component {
 
         </div>
 
-        <div className='ArticleNavigator-unsaved'>
+        <div className={'ArticleNavigator-unsaved' + (hasModified ? '' : ' hide')}>
           <div className='ArticleNavigator-unsaved-header'>Work in progress</div>
           <div className='ArticleNavigator-unsaved-list'>
-            {modifiedElements.length > 0
-              ? modifiedElements
-              : (
-                <div className='ArticleNavigator-unsaved-list-empty'>Empty list</div>
-              )
-            }
+            {modifiedElements}
           </div>
           <div className='ArticleNavigator-unsaved-control'>
             <button onClick={e => this.handleSaveAllClick()} className='ArticleNavigator-unsaved-control-save-all-button' disabled={modifiedElements.length === 0}>Save all</button>
           </div>
         </div>
 
-        <div className='ArticleNavigator-folders'>
+        <div className={'ArticleNavigator-folders' + (hasModified ? '' : ' expand')}>
           <div className='ArticleNavigator-folders-header'>
             <div className='title'>Folders</div>
             <button onClick={e => this.handleNewFolderButton(e)} className='addBtn'>
diff --git a/browser/styles/main/ArticleDetail.styl b/browser/styles/main/ArticleDetail.styl
index 5340d88d..46ea1f4e 100644
--- a/browser/styles/main/ArticleDetail.styl
+++ b/browser/styles/main/ArticleDetail.styl
@@ -6,6 +6,7 @@ infoButton()
   cursor pointer
   height 33px
   width 33px
+  line-height 33px
   margin-right 5px
   font-size 18px
   color inactiveTextColor
@@ -67,8 +68,10 @@ infoButton()
         color brandColor
     .ArticleDetail-info-control
       float right
+      clearfix
       .ShareButton
-        display inline-block
+        display block
+        float left
       &>button, .ShareButton-open-button
         infoButton()
         .tooltip
@@ -78,8 +81,35 @@ infoButton()
           .tooltip
             opacity 1
       &>button
-        &:nth-child(2) .tooltip
+        float left
+        &:nth-child(1) .tooltip
           margin-left -65px
+      .ArticleDetail-info-control-save
+        float left
+        width 80px
+        margin-right 5px
+        overflow hidden
+        transition width 0.15s ease-in-out
+        border-radius 16.5px
+        &.hide
+          width 0px
+          opacity 0.2
+      .ArticleDetail-info-control-save-button
+        infoButton()
+        background-color brandColor
+        color white
+        font-size 12px
+        width 100%
+        border 1px solid brandBorderColor
+        white-space nowrap
+        .fa
+          font-size 18px
+        &:hover
+          color white
+          background-color lighten(brandColor, 15%)
+        &:focus
+          color white
+          background-color lighten(brandColor, 15%)
       .ShareButton-open-button .tooltip
         margin-left -40px
       .ShareButton-dropdown
diff --git a/browser/styles/main/ArticleNavigator.styl b/browser/styles/main/ArticleNavigator.styl
index 43eaebca..d7cc11d2 100644
--- a/browser/styles/main/ArticleNavigator.styl
+++ b/browser/styles/main/ArticleNavigator.styl
@@ -61,12 +61,14 @@ articleCount = #999
     top 100px
     width 100%
     height 225px
+    transition opacity 0.2s ease-in-out
+    &.hide
+      opacity 0.2
     .ArticleNavigator-unsaved-header
       border-bottom 1px solid alpha(borderColor, 0.5)
       padding-bottom 5px
       clearfix()
       position relative
-      z-index 30
       padding-left 10px
       font-size 18px
       line-height 22px
@@ -140,6 +142,10 @@ articleCount = #999
     absolute bottom
     top 365px
     width 100%
+    transition top 0.15s ease-in-out
+    background-color articleNavBgColor
+    &.expand
+      top 100px
     .ArticleNavigator-folders-header
       border-bottom 1px solid alpha(borderColor, 0.5)
       padding-bottom 5px