From 044a6eaadae59ab91edaef8268f655f1e32a9a4e Mon Sep 17 00:00:00 2001
From: danicatalan <danicatalan@hotmail.com>
Date: Thu, 25 Aug 2016 17:40:42 +0200
Subject: [PATCH 1/4] fix footer height

---
 app/react/Layout/scss/sidepanel.scss | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/react/Layout/scss/sidepanel.scss b/app/react/Layout/scss/sidepanel.scss
index aa8fc66dd1..47d0cb3859 100644
--- a/app/react/Layout/scss/sidepanel.scss
+++ b/app/react/Layout/scss/sidepanel.scss
@@ -137,6 +137,7 @@ $c-sidebar: $c-white;
   position: absolute;
   bottom: 0;
   width: 100%;
+  padding-top: 0;
   text-align: center;
   background: linear-gradient(transparent 0%, $c-white 70%, $c-white 100%);
 

From 1d6ca8c583ce0597b7638227958a6f2f8ceed278 Mon Sep 17 00:00:00 2001
From: danicatalan <danicatalan@hotmail.com>
Date: Thu, 25 Aug 2016 17:54:26 +0200
Subject: [PATCH 2/4] move header into body on entities (for scrolling
 purposes)

---
 app/react/App/scss/modules/_entity.scss       | 11 +++++++++--
 app/react/Entities/components/EntityViewer.js | 15 ++++++++++-----
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/app/react/App/scss/modules/_entity.scss b/app/react/App/scss/modules/_entity.scss
index 026bc12c88..186009a9f0 100644
--- a/app/react/App/scss/modules/_entity.scss
+++ b/app/react/App/scss/modules/_entity.scss
@@ -10,8 +10,15 @@
     top: 0 !important;
   }
   .sidepanel-header {
-    border: 0;
-    padding-bottom: 0;
+    display: none;
+  }
+  .sidepanel-body {
+    .item-name {
+      padding: 30px 30px 0;
+    }
+    .item-type {
+      margin-left: 30px;
+    }
   }
   dt {
     width: 25%;
diff --git a/app/react/Entities/components/EntityViewer.js b/app/react/Entities/components/EntityViewer.js
index 077e1901c9..2985339bfc 100644
--- a/app/react/Entities/components/EntityViewer.js
+++ b/app/react/Entities/components/EntityViewer.js
@@ -65,11 +65,7 @@ export class EntityViewer extends Component {
         <aside className="side-panel entity-metadata">
         <ShowIf if={!entityBeingEdited}>
           <div className="sidepanel-header">
-            <h1 className="item-name">{entity.title}</h1>
-            <span className="item-type item-type-2">
-              <i className="item-type__icon fa fa-bank"></i>
-              <span className="item-type__name">{entity.documentType}</span>
-            </span>
+
           </div>
         </ShowIf>
             <div className="sidepanel-footer">
@@ -97,6 +93,15 @@ export class EntityViewer extends Component {
               </NeedAuthorization>
             </div>
           <div className="sidepanel-body">
+            <ShowIf if={!entityBeingEdited}>
+              <h1 className="item-name">{entity.title}</h1>
+            </ShowIf>
+            <ShowIf if={!entityBeingEdited}>
+              <span className="item-type item-type-2">
+                <i className="item-type__icon fa fa-bank"></i>
+                <span className="item-type__name">{entity.documentType}</span>
+              </span>
+            </ShowIf>
           {(() => {
             if (entityBeingEdited) {
               return <EntityForm/>;

From 24a49b62430850cc872741bfbae380e5f0d8260b Mon Sep 17 00:00:00 2001
From: danicatalan <danicatalan@hotmail.com>
Date: Thu, 25 Aug 2016 18:41:37 +0200
Subject: [PATCH 3/4] improve sidebar tabs

---
 app/react/App/scss/layout/_nav.scss           | 17 ++++++----
 app/react/Entities/components/EntityViewer.js | 10 +++++-
 app/react/Layout/scss/sidepanel.scss          | 33 ++++++++++++-------
 .../Library/components/ViewMetadataPanel.js   | 10 ++++--
 .../Viewer/components/ViewMetadataPanel.js    | 16 ++++++---
 5 files changed, 59 insertions(+), 27 deletions(-)

diff --git a/app/react/App/scss/layout/_nav.scss b/app/react/App/scss/layout/_nav.scss
index c67ed56c82..d912086da0 100644
--- a/app/react/App/scss/layout/_nav.scss
+++ b/app/react/App/scss/layout/_nav.scss
@@ -61,27 +61,30 @@ nav ul {
 .nav-tabs>li {
   display: inline-block;
   float: none;
-  width: 100%;
-  max-width: 33%;
+  min-width: 54px
 }
 
 .nav-tabs>li>a, .nav-tabs>li>div {
   cursor:pointer;
-  line-height: 40px;
-  padding: 0;
-  margin-top: 5px;
+  line-height: 54px;
+  padding: 0 15px;
   background-color: transparent;
   color: $c-grey-dark;
   border: 0;
   border-bottom: 2px solid transparent;
   border-radius: 0;
+  .fa {
+    font-size: 16px;
+    position: relative;
+    top: 3px;
+  }
 }
 
 .nav-tabs>li>a:focus, .nav-tabs>li>div:focus, .nav-tabs>li>a:hover, .nav-tabs>li>div:hover {
   background-color: transparent;
   color: $c-black;
   border: 0;
-  border-bottom: 3px solid $c-grey-dark;
+  border-bottom: 2px solid $c-grey-dark;
 }
 
 
@@ -91,5 +94,5 @@ nav ul {
   background-color: transparent;
   color: $c-black;
   border: 0;
-  border-bottom: 3px solid $c-primary;
+  border-bottom: 2px solid $c-primary;
 }
diff --git a/app/react/Entities/components/EntityViewer.js b/app/react/Entities/components/EntityViewer.js
index 2985339bfc..09a408a9a0 100644
--- a/app/react/Entities/components/EntityViewer.js
+++ b/app/react/Entities/components/EntityViewer.js
@@ -113,7 +113,15 @@ export class EntityViewer extends Component {
         </aside>
         <aside className="side-panel entity-connections">
           <div className="sidepanel-header">
-            <h1>Connections ({references.length})</h1>
+            <ul className="nav nav-tabs">
+              <li>
+                <div className="tab-link tab-link-active">
+                  <i className="fa fa-sitemap"></i>
+                  <span className="connectionsNumber">{references.length}</span>
+                </div>
+              </li>
+            </ul>
+            &nbsp;
           </div>
           <div className="sidepanel-body">
             <div className="item-group">
diff --git a/app/react/Layout/scss/sidepanel.scss b/app/react/Layout/scss/sidepanel.scss
index 47d0cb3859..e9e667a71a 100644
--- a/app/react/Layout/scss/sidepanel.scss
+++ b/app/react/Layout/scss/sidepanel.scss
@@ -56,15 +56,13 @@ $c-sidebar: $c-white;
 
   .fa-close {
     position: absolute;
-    top: 15px;
-    right: 15px;
-    font-size: 18px;
-    color: $c-grey;
-    cursor: pointer;
-    position: absolute;
-    top: 30px;
-    right: 30px;
-    font-size: 18px;
+    top: 0;
+    right: 0;
+    width: 54px;
+    height: 54px;
+    line-height: 54px;
+    text-align: center;
+    font-size: $f-size-lg;
     color: $c-grey;
     cursor: pointer;
     z-index: 2;
@@ -112,7 +110,7 @@ $c-sidebar: $c-white;
 .sidepanel-header,
 .sidepanel-footer {
   background: $c-sidebar;
-  padding: 30px;
+  padding: 18px 30px;
   z-index: 1;
 }
 
@@ -129,8 +127,19 @@ $c-sidebar: $c-white;
   }
 
   .nav-tabs {
-    margin: -15px;
-    margin-top: 0;
+    position: absolute;
+    left: 0;
+    top: 0;
+    height: 54px;
+    right: 54px;
+    border: 0;
+  }
+  .connectionsNumber {
+    position: relative;
+    top: 2px;
+    font-size: 11px;
+    line-height: 1em;
+    margin-left: 5px;
   }
 }
 .sidepanel-footer {
diff --git a/app/react/Library/components/ViewMetadataPanel.js b/app/react/Library/components/ViewMetadataPanel.js
index c71aac53d0..a53b99ab75 100644
--- a/app/react/Library/components/ViewMetadataPanel.js
+++ b/app/react/Library/components/ViewMetadataPanel.js
@@ -58,8 +58,14 @@ export class ViewMetadataPanel extends Component {
     return (
       <SidePanel open={this.props.open}>
         <div className="sidepanel-header">
-          <h1>Metadata</h1>
-          <i className="fa fa-close close-modal" onClick={this.close.bind(this)}/>
+            <ul className="nav nav-tabs">
+              <li>
+                <div className="tab-link tab-link-active">
+                  <i className="fa fa-info-circle"></i>
+                </div>
+              </li>
+            </ul>
+            <i className="fa fa-close close-modal" onClick={this.close.bind(this)}/>&nbsp;
         </div>
         <div className="sidepanel-footer">
           <NeedAuthorization>
diff --git a/app/react/Viewer/components/ViewMetadataPanel.js b/app/react/Viewer/components/ViewMetadataPanel.js
index f13d09251b..645020922a 100644
--- a/app/react/Viewer/components/ViewMetadataPanel.js
+++ b/app/react/Viewer/components/ViewMetadataPanel.js
@@ -53,8 +53,7 @@ export class ViewMetadataPanel extends Component {
     return (
       <SidePanel open={this.props.open} className="metadata-sidepanel">
         <div className="sidepanel-header">
-          <h1>Metadata</h1>
-          <i className="fa fa-close close-modal" onClick={this.close.bind(this)}/>
+          <i className="fa fa-close close-modal" onClick={this.close.bind(this)}/>&nbsp;
           <Tabs selectedTab={this.props.tab || 'metadata'}
             handleSelect={(tab) => {
               this.props.showTab(tab);
@@ -62,13 +61,20 @@ export class ViewMetadataPanel extends Component {
           >
             <ul className="nav nav-tabs">
               <li>
-                <TabLink to="toc">Table of contents</TabLink>
+                <TabLink to="toc">
+                  <i className="fa fa-indent"></i>
+                </TabLink>
               </li>
               <li>
-                <TabLink to="metadata" default>Metadata</TabLink>
+                <TabLink to="metadata" default>
+                  <i className="fa fa-info-circle"></i>
+                </TabLink>
               </li>
               <li>
-                <TabLink to="connections">Connections&nbsp;({this.props.numberOfReferences})</TabLink>
+                <TabLink to="connections">
+                  <i className="fa fa-sitemap"></i>
+                  <span className="connectionsNumber">{this.props.numberOfReferences}</span>
+                </TabLink>
               </li>
             </ul>
           </Tabs>

From 19f3d8470e7dcab930362f1750c780e7072012ee Mon Sep 17 00:00:00 2001
From: danicatalan <danicatalan@hotmail.com>
Date: Thu, 25 Aug 2016 18:45:28 +0200
Subject: [PATCH 4/4] define icon for open document

---
 app/react/App/scss/layout/_header.scss    | 5 ++---
 app/react/Library/components/Doc.js       | 2 +-
 app/react/Library/components/SearchBar.js | 2 +-
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/app/react/App/scss/layout/_header.scss b/app/react/App/scss/layout/_header.scss
index e936843514..f215b5bf27 100644
--- a/app/react/App/scss/layout/_header.scss
+++ b/app/react/App/scss/layout/_header.scss
@@ -292,11 +292,10 @@ header {
     }
   }
 
-	.fa-arrow-left {
+	.fa-external-link {
 		position: absolute;
-		top: 8px;
+		top: 10px;
 		right: 15px;
-		transform: rotate(45deg);
 	}
 
 }
diff --git a/app/react/Library/components/Doc.js b/app/react/Library/components/Doc.js
index a0f0443a46..867b6dcd20 100644
--- a/app/react/Library/components/Doc.js
+++ b/app/react/Library/components/Doc.js
@@ -57,7 +57,7 @@ export class Doc extends Component {
         </div>
         <ItemFooter>
           <Link to={documentViewUrl} className="item-shortcut">
-            <i className="fa fa-chevron-right"></i>
+            <i className="fa fa-external-link"></i>
           </Link>
         </ItemFooter>
       </RowList.Item>
diff --git a/app/react/Library/components/SearchBar.js b/app/react/Library/components/SearchBar.js
index 61a719b11a..e499d4c04d 100644
--- a/app/react/Library/components/SearchBar.js
+++ b/app/react/Library/components/SearchBar.js
@@ -71,7 +71,7 @@ export class SearchBar extends Component {
               return <p key={index}>
                 <Link to={documentViewUrl}>
                   <span dangerouslySetInnerHTML={{__html: suggestion.title}}/>
-                  <i className="fa fa-arrow-left">
+                  <i className="fa fa-external-link">
                   </i>
                 </Link>
               </p>;