From c2eaac67295b092ffc91dbd9a35ededfed057d99 Mon Sep 17 00:00:00 2001
From: jessop <jessop@lbry.com>
Date: Thu, 23 Jan 2020 13:44:55 -0500
Subject: [PATCH] review changes

---
 .lintstagedrc.json                         |  1 +
 lbrytv/component/{ad/index.jsx => ads.jsx} | 16 ++++++++++------
 ui/component/sideNavigation/view.jsx       |  2 +-
 ui/scss/component/_navigation.scss         |  2 --
 ui/scss/init/_vars.scss                    |  1 -
 5 files changed, 12 insertions(+), 10 deletions(-)
 rename lbrytv/component/{ad/index.jsx => ads.jsx} (67%)

diff --git a/.lintstagedrc.json b/.lintstagedrc.json
index 38c99b105..b3430d6a4 100644
--- a/.lintstagedrc.json
+++ b/.lintstagedrc.json
@@ -1,6 +1,7 @@
 {
   "linters": {
     "ui/**/*.{js,jsx,scss,json}": ["prettier --write", "git add"],
+    "lbrytv/component/*.{js,jsx,scss,json}": ["prettier --write", "git add"],
     "ui/**/*.{js,jsx}": ["eslint", "flow focus-check --color always", "git add"]
   }
 }
diff --git a/lbrytv/component/ad/index.jsx b/lbrytv/component/ads.jsx
similarity index 67%
rename from lbrytv/component/ad/index.jsx
rename to lbrytv/component/ads.jsx
index 01b62663c..e7162f434 100644
--- a/lbrytv/component/ad/index.jsx
+++ b/lbrytv/component/ads.jsx
@@ -15,19 +15,23 @@ function Extra(props) {
       document.body.removeChild(script);
       // if user navigates too rapidly, <style> tags can build up
       if (document.body.getElementsByTagName('style').length) {
-          document.body.getElementsByTagName('style')[0].remove();
+        document.body.getElementsByTagName('style')[0].remove();
       }
     };
   }, []);
 
   return (
     <>
-      <div><p>Ads</p></div>
-      <div>
-      <div id="rc-widget-0a74cf" data-rc-widget data-widget-host="habitat" data-endpoint="//trends.revcontent.com" data-widget-id="117427" />
-      </div>
+      <p>Ads</p>
+      <div
+        id="rc-widget-0a74cf"
+        data-rc-widget
+        data-widget-host="habitat"
+        data-endpoint="//trends.revcontent.com"
+        data-widget-id="117427"
+      />
     </>
   );
-};
+}
 
 export default Extra;
diff --git a/ui/component/sideNavigation/view.jsx b/ui/component/sideNavigation/view.jsx
index c3583acb4..aeb7550b8 100644
--- a/ui/component/sideNavigation/view.jsx
+++ b/ui/component/sideNavigation/view.jsx
@@ -8,7 +8,7 @@ import Tag from 'component/tag';
 import StickyBox from 'react-sticky-box/dist/esnext';
 import Spinner from 'component/spinner';
 import usePersistedState from 'effects/use-persisted-state';
-import Ad from 'lbrytv/component/ad';
+import Ad from 'lbrytv/component/ads';
 
 const SHOW_CHANNELS = 'SHOW_CHANNELS';
 const SHOW_TAGS = 'SHOW_TAGS';
diff --git a/ui/scss/component/_navigation.scss b/ui/scss/component/_navigation.scss
index 7ca145f6b..b1d081785 100644
--- a/ui/scss/component/_navigation.scss
+++ b/ui/scss/component/_navigation.scss
@@ -15,7 +15,6 @@
 
 .navigation--placeholder {
   @extend .navigation;
-  padding: var(--spacing-quarter) 0rem;
   border-radius: var(--card-radius);
   position: relative;
   margin: auto;
@@ -23,7 +22,6 @@
   p {
     font-size: var(--font-xsmall);
     color: var(--color-text-subtitle);
-    padding: var(--spacing-quarter);
   }
 }
 
diff --git a/ui/scss/init/_vars.scss b/ui/scss/init/_vars.scss
index 77dd4e4b2..92bfbaaa1 100644
--- a/ui/scss/init/_vars.scss
+++ b/ui/scss/init/_vars.scss
@@ -11,7 +11,6 @@ $breakpoint-small: 900px;
   --mac-titlebar-height: 1.5rem;
   --mobile: 600px;
   --side-nav-width: 170px;
-  --spacing-quarter: calc(1rem / 4);
   --spacing-miniscule: calc(2rem / 5);
   --spacing-xsmall: calc(2rem / 4);
   --spacing-small: calc(2rem / 3);