From 8bebb12ea648d0d4624a0cd83ebadcc8a9be1e40 Mon Sep 17 00:00:00 2001
From: Janne Koivistoinen <janne@sharetribe.com>
Date: Fri, 27 Oct 2017 13:57:54 +0300
Subject: [PATCH] reviews fixed

---
 docs/static-pages.md                          | 28 ++++-----
 src/components/Footer/Footer.js               |  4 +-
 .../LayoutSingleColumn/LayoutSingleColumn.css |  1 +
 src/containers/AboutPage/AboutPage.css        |  6 +-
 src/containers/AboutPage/AboutPage.js         | 61 ++++++++++++-------
 src/routeConfiguration.js                     |  2 +-
 6 files changed, 56 insertions(+), 46 deletions(-)

diff --git a/docs/static-pages.md b/docs/static-pages.md
index 8ac165a6..58fce874 100644
--- a/docs/static-pages.md
+++ b/docs/static-pages.md
@@ -50,14 +50,12 @@ const AboutPage = () => {
     <StaticPage
       className={css.root}
       title="About"
-      schema={`
-        {
-          "@context": "http://schema.org",
-          "@type": "AboutPage",
-          "description": "Description of this page",
-          "name": "About page",
-        }
-      `}
+      schema={{
+        "@context": "http://schema.org",
+        "@type": "AboutPage",
+        "description": "Description of this page",
+        "name": "About page",
+      }}
     >
       <LayoutSingleColumn>
         <LayoutWrapperTopbar>
@@ -136,14 +134,12 @@ In the template above we are using StaticPage component with some attributes:
     <StaticPage
       className={css.root}
       title="About"
-      schema={`
-        {
-          "@context": "http://schema.org",
-          "@type": "AboutPage",
-          "description": "Description of this page",
-          "name": "About page",
-        }
-      `}
+      schema={{
+        "@context": "http://schema.org",
+        "@type": "AboutPage",
+        "description": "Description of this page",
+        "name": "About page",
+      }}
     >
 ```
 
diff --git a/src/components/Footer/Footer.js b/src/components/Footer/Footer.js
index 6a3fe8f5..7b6d7191 100644
--- a/src/components/Footer/Footer.js
+++ b/src/components/Footer/Footer.js
@@ -94,7 +94,7 @@ const Footer = props => {
                 </NamedLink>
               </li>
               <li className={css.listItem}>
-                <NamedLink name="LandingPage" className={css.link}>
+                <NamedLink name="AboutPage" className={css.link}>
                   <FormattedMessage id="Footer.toAboutPage" />
                 </NamedLink>
               </li>
@@ -109,7 +109,7 @@ const Footer = props => {
                 </NamedLink>
               </li>
               <li className={css.listItem}>
-                <NamedLink name="LandingPage" className={css.link}>
+                <NamedLink name="AboutPage" to={{ hash: '#contact' }} className={css.link}>
                   <FormattedMessage id="Footer.toContactPage" />
                 </NamedLink>
               </li>
diff --git a/src/components/LayoutSingleColumn/LayoutSingleColumn.css b/src/components/LayoutSingleColumn/LayoutSingleColumn.css
index f936640e..1fffb6d2 100644
--- a/src/components/LayoutSingleColumn/LayoutSingleColumn.css
+++ b/src/components/LayoutSingleColumn/LayoutSingleColumn.css
@@ -13,4 +13,5 @@
   /* Expand to the full remaining width of the viewport */
   display: flex;
   flex-direction: column;
+  width: 100%;
 }
diff --git a/src/containers/AboutPage/AboutPage.css b/src/containers/AboutPage/AboutPage.css
index 62cc35d4..fe3b2a37 100644
--- a/src/containers/AboutPage/AboutPage.css
+++ b/src/containers/AboutPage/AboutPage.css
@@ -1,8 +1,5 @@
 @import '../../marketplace.css';
 
-.root {
-}
-
 .pageTitle {
   text-align: center;
 }
@@ -32,7 +29,6 @@
   @media (--viewportMedium) {
     flex-wrap: nowrap;
   }
-
 }
 
 .contentSide {
@@ -51,4 +47,4 @@
   @media (--viewportMedium) {
     max-width: 650px;
   }
-}
\ No newline at end of file
+}
diff --git a/src/containers/AboutPage/AboutPage.js b/src/containers/AboutPage/AboutPage.js
index 9aaa0927..9a6440dd 100644
--- a/src/containers/AboutPage/AboutPage.js
+++ b/src/containers/AboutPage/AboutPage.js
@@ -6,7 +6,7 @@ import {
   LayoutWrapperMain,
   LayoutWrapperFooter,
   Footer,
-  ExternalLink
+  ExternalLink,
 } from '../../components';
 
 import css from './AboutPage.css';
@@ -15,19 +15,15 @@ import image from './about-us.jpg';
 const AboutPage = () => {
   return (
     <StaticPage
-      className={css.root}
       title="About Us"
-      schema={`
-        {
-          "@context": "http://schema.org",
-          "@type": "AboutPage",
-          "description": "Description of this page",
-          "name": "About page",
-        }
-      `}
+      schema={{
+        '@context': 'http://schema.org',
+        '@type': 'AboutPage',
+        description: 'About Saunatime',
+        name: 'About page',
+      }}
     >
       <LayoutSingleColumn>
-
         <LayoutWrapperTopbar>
           <TopbarContainer />
         </LayoutWrapperTopbar>
@@ -37,35 +33,56 @@ const AboutPage = () => {
           <img className={css.coverImage} src={image} alt="My first ice cream." />
 
           <div className={css.contentWrapper}>
-
             <div className={css.contentSide}>
               <p>"We built Saunatime because we didn't have a sauna in our apartment building."</p>
             </div>
 
             <div className={css.contentMain}>
-              <h2>Fusce dapibus, tellus ac cursus commodo, tortor mauris condimentum nibh, ut fermentum massa justo sit amet risus. Praesent commodo cursus magna, vel scelerisque nisl consectetur et. Cras justo odio, dapibus ac facilisis in, egestas eget quam.</h2>
+              <h2>
+                Fusce dapibus, tellus ac cursus commodo, tortor mauris condimentum nibh, ut
+                fermentum massa justo sit amet risus. Praesent commodo cursus magna, vel scelerisque
+                nisl consectetur et. Cras justo odio, dapibus ac facilisis in, egestas eget quam.
+              </h2>
 
               <h3>Tortor mauris condimentum</h3>
 
-              <p>Fusce dapibus, tellus ac cursus commodo, tortor mauris condimentum nibh, ut fermentum massa justo sit amet risus. Praesent commodo cursus magna, vel scelerisque nisl consectetur et. Cras justo odio, dapibus ac facilisis in, egestas eget quam.</p>
+              <p>
+                Fusce dapibus, tellus ac cursus commodo, tortor mauris condimentum nibh, ut
+                fermentum massa justo sit amet risus. Praesent commodo cursus magna, vel scelerisque
+                nisl consectetur et. Cras justo odio, dapibus ac facilisis in, egestas eget quam.
+              </p>
 
-              <p>Integer posuere erat a ante venenatis dapibus posuere velit aliquet. Cras mattis consectetur purus sit amet fermentum. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus.</p>
+              <p>
+                Integer posuere erat a ante venenatis dapibus posuere velit aliquet. Cras mattis
+                consectetur purus sit amet fermentum. Cum sociis natoque penatibus et magnis dis
+                parturient montes, nascetur ridiculus mus.
+              </p>
 
-              <h3>Get in touch</h3>
-              <p>We are happy to help you in anything you have in your mind.?Best way to reach us is by emailing us at <ExternalLink href="mailto:hello@saunatime.io?Subject=Hello%20friends">hello@saunatime.io</ExternalLink>.</p>
-              <p>You can also checkout our <ExternalLink href="https://www.facebook.com/Sharetribe/">Facebook</ExternalLink> and <ExternalLink href="https://twitter.com/sharetribe">Twitter</ExternalLink>.</p>
+              <h3 id="contact">Get in touch</h3>
+              <p>
+                We are happy to help you in anything you have in your mind.?Best way to reach us is
+                by emailing us at{' '}
+                <ExternalLink href="mailto:hello@saunatime.io?Subject=Hello%20friends">
+                  hello@saunatime.io
+                </ExternalLink>.
+              </p>
+              <p>
+                You can also checkout our{' '}
+                <ExternalLink href="https://www.facebook.com/Sharetribe/">
+                  Facebook
+                </ExternalLink>{' '}
+                and <ExternalLink href="https://twitter.com/sharetribe">Twitter</ExternalLink>.
+              </p>
             </div>
           </div>
-
         </LayoutWrapperMain>
 
         <LayoutWrapperFooter>
           <Footer />
         </LayoutWrapperFooter>
-
       </LayoutSingleColumn>
     </StaticPage>
   );
-}
+};
 
-export default AboutPage;
\ No newline at end of file
+export default AboutPage;
diff --git a/src/routeConfiguration.js b/src/routeConfiguration.js
index ddfdb089..6480c90a 100644
--- a/src/routeConfiguration.js
+++ b/src/routeConfiguration.js
@@ -43,7 +43,7 @@ const routeConfiguration = () => {
     {
       path: '/',
       name: 'LandingPage',
-      component: props => <LandingPage {...props} />
+      component: props => <LandingPage {...props} />,
     },
     {
       path: '/about',