From f21c237d62952224987170f655cc85082a8758ff Mon Sep 17 00:00:00 2001
From: Tom Moor <tom.moor@gmail.com>
Date: Sun, 24 Sep 2017 22:38:53 -0700
Subject: [PATCH] Fixes new document menu not working Fixes styling when drag
 and dropping into sidebar

---
 .../Layout/components/SidebarCollections.js   | 21 ++++++++++---------
 frontend/styles/base.css                      |  8 +++++++
 2 files changed, 19 insertions(+), 10 deletions(-)

diff --git a/frontend/components/Layout/components/SidebarCollections.js b/frontend/components/Layout/components/SidebarCollections.js
index 9bd931514..3544e3040 100644
--- a/frontend/components/Layout/components/SidebarCollections.js
+++ b/frontend/components/Layout/components/SidebarCollections.js
@@ -24,16 +24,11 @@ type Props = {
   ui: UiStore,
 };
 
-const activeStyle = {
-  color: color.black,
-  background: color.slateDark,
-};
-
 @observer class SidebarCollections extends React.PureComponent {
   props: Props;
 
   render() {
-    const { collections, activeDocument, ui } = this.props;
+    const { history, collections, activeDocument, ui } = this.props;
 
     return (
       <Flex column>
@@ -41,6 +36,7 @@ const activeStyle = {
         {collections.data.map(collection => (
           <CollectionLink
             key={collection.id}
+            history={history}
             collection={collection}
             activeDocument={activeDocument}
             ui={ui}
@@ -66,13 +62,14 @@ const activeStyle = {
   };
 
   render() {
-    const { collection, activeDocument, ui } = this.props;
+    const { history, collection, activeDocument, ui } = this.props;
+
     return (
       <DropToImport
         key={collection.id}
         history={history}
         collectionId={collection.id}
-        activeStyle={activeStyle}
+        activeClassName="activeDropZone"
         onMouseEnter={this.handleHover}
         onMouseLeave={this.handleBlur}
       >
@@ -83,6 +80,7 @@ const activeStyle = {
             {(this.isHovering || this.menuOpen) &&
               <CollectionAction>
                 <CollectionMenu
+                  history={history}
                   collection={collection}
                   onShow={() => (this.menuOpen = true)}
                   onClose={() => (this.menuOpen = false)}
@@ -93,6 +91,7 @@ const activeStyle = {
             collection.documents.map(document => (
               <DocumentLink
                 key={document.id}
+                history={history}
                 document={document}
                 activeDocument={activeDocument}
                 depth={0}
@@ -106,12 +105,13 @@ const activeStyle = {
 
 type DocumentLinkProps = {
   document: NavigationNode,
+  history: Object,
   activeDocument: ?Document,
   depth: number,
 };
 
 const DocumentLink = observer((props: DocumentLinkProps) => {
-  const { document, activeDocument, depth } = props;
+  const { history, document, activeDocument, depth } = props;
   const canDropToImport = depth === 0;
 
   return (
@@ -120,7 +120,7 @@ const DocumentLink = observer((props: DocumentLinkProps) => {
         <DropToImport
           history={history}
           documentId={document.id}
-          activeStyle={activeStyle}
+          activeClassName="activeDropZone"
         >
           <SidebarLink to={document.url}>{document.title}</SidebarLink>
         </DropToImport>}
@@ -137,6 +137,7 @@ const DocumentLink = observer((props: DocumentLinkProps) => {
             document.children.map(childDocument => (
               <DocumentLink
                 key={childDocument.id}
+                history={history}
                 document={childDocument}
                 depth={depth + 1}
               />
diff --git a/frontend/styles/base.css b/frontend/styles/base.css
index 52f316ae8..27eabf039 100644
--- a/frontend/styles/base.css
+++ b/frontend/styles/base.css
@@ -129,3 +129,11 @@ hr {
   padding: 1em;
   border-radius: 0.25em;
 }
+
+.activeDropZone {
+  background: #4E5C6E;
+}
+
+.activeDropZone a {
+  color: #FFF !important;
+}