From 6756f09a77c0be57eb88e5cff13b6635fccbf58b Mon Sep 17 00:00:00 2001
From: Khoa Tran <khoatran@temando.com>
Date: Tue, 6 Jun 2017 01:41:07 +1000
Subject: [PATCH] fix issue with scroll synchronisation

---
 src/components/Indicator/Indicator.js         |  6 ++--
 src/components/Indicator/Indicator.scss       |  4 +--
 src/components/Navigation/Navigation.js       |  6 ++--
 .../NavigationMethod/NavigationMethod.js      |  9 +++--
 .../NavigationMethod/NavigationMethod.scss    |  8 +++++
 src/components/NavigationTag/NavigationTag.js | 33 ++++++++++++++-----
 .../NavigationTag/NavigationTag.scss          |  8 +++++
 .../ServiceContainer/ServiceContainer.js      | 18 ++++------
 src/handlers/BaseHandler.js                   |  2 +-
 9 files changed, 61 insertions(+), 33 deletions(-)

diff --git a/src/components/Indicator/Indicator.js b/src/components/Indicator/Indicator.js
index 5f15e59..e67a1b4 100644
--- a/src/components/Indicator/Indicator.js
+++ b/src/components/Indicator/Indicator.js
@@ -8,11 +8,11 @@ const arrow = require('./arrow.png')
 
 export default class Indicator extends PureComponent {
   render () {
-    const { status, className } = this.props
+    const { direction, className } = this.props
 
     return (
       <span className={classNames('indicator', className, {
-        [`indicator--${status}`]: status
+        [`indicator--${direction}`]: direction
       })}>
         <img src={arrow} alt='' title='arrow' />
       </span>
@@ -21,6 +21,6 @@ export default class Indicator extends PureComponent {
 }
 
 Indicator.propTypes = {
-  status: PropTypes.string,
+  direction: PropTypes.string,
   className: PropTypes.string
 }
diff --git a/src/components/Indicator/Indicator.scss b/src/components/Indicator/Indicator.scss
index 8825947..65c715f 100644
--- a/src/components/Indicator/Indicator.scss
+++ b/src/components/Indicator/Indicator.scss
@@ -11,10 +11,10 @@
   width: 20px;
 }
 
-.indicator--open img {
+.indicator--down img {
   transform: rotate(180deg);
 }
 
 .indicator--right img {
   transform: rotate(-90deg);
-}
\ No newline at end of file
+}
diff --git a/src/components/Navigation/Navigation.js b/src/components/Navigation/Navigation.js
index 0608da3..5880258 100644
--- a/src/components/Navigation/Navigation.js
+++ b/src/components/Navigation/Navigation.js
@@ -31,9 +31,9 @@ export default class Navigation extends Component {
     return (
       <nav className='nav'>
         {navigation && navigation.map((tag) => {
-          let status
+          let shouldBeExpanded = false
           if (expandedTags.includes(tag.title)) {
-            status = 'right'
+            shouldBeExpanded = true
           }
 
           return (
@@ -41,7 +41,7 @@ export default class Navigation extends Component {
               key={tag.title}
               title={tag.title}
               methods={tag.methods}
-              status={status}
+              shouldBeExpanded={shouldBeExpanded}
               onClick={this.onClick}
               location={location}
             />
diff --git a/src/components/NavigationMethod/NavigationMethod.js b/src/components/NavigationMethod/NavigationMethod.js
index d475f48..0864917 100644
--- a/src/components/NavigationMethod/NavigationMethod.js
+++ b/src/components/NavigationMethod/NavigationMethod.js
@@ -6,12 +6,14 @@ import './NavigationMethod.scss'
 
 export default class NavigationMethod extends PureComponent {
   render () {
-    const { method, isActive } = this.props
+    const { method, isActive, isOpen } = this.props
 
     return (
       <a
         className={classNames('nav-method', {
-          active: isActive
+          active: isActive,
+          open: isOpen,
+          closed: !isOpen
         })}
         href={`#${method.link}`}
       >
@@ -24,5 +26,6 @@ export default class NavigationMethod extends PureComponent {
 
 NavigationMethod.propTypes = {
   method: PropTypes.object,
-  isActive: PropTypes.bool
+  isActive: PropTypes.bool,
+  isOpen: PropTypes.bool
 }
diff --git a/src/components/NavigationMethod/NavigationMethod.scss b/src/components/NavigationMethod/NavigationMethod.scss
index d7af439..4114376 100644
--- a/src/components/NavigationMethod/NavigationMethod.scss
+++ b/src/components/NavigationMethod/NavigationMethod.scss
@@ -10,6 +10,14 @@
   &.active {
     background-color: $grey-dark;
   }
+
+  &.open {
+    display: block;
+  }
+
+  &.closed {
+    display: none;
+  }
 }
 
 .method-title {
diff --git a/src/components/NavigationTag/NavigationTag.js b/src/components/NavigationTag/NavigationTag.js
index a80e76b..3e8b4fb 100644
--- a/src/components/NavigationTag/NavigationTag.js
+++ b/src/components/NavigationTag/NavigationTag.js
@@ -1,6 +1,5 @@
 import React, { Component } from 'react'
 import PropTypes from 'prop-types'
-
 import Indicator from '../Indicator/Indicator'
 import NavigationMethod from '../NavigationMethod/NavigationMethod'
 
@@ -15,9 +14,9 @@ export default class NavigationTag extends Component {
 
   shouldComponentUpdate (nextProps, nextState) {
     const isHashDiff = this.props.location.hash !== nextProps.location.hash
-    const isStatusDiff = this.props.status !== nextProps.status
+    const isExpandedStatusDiff = this.props.shouldBeExpanded !== nextProps.shouldBeExpanded
 
-    return isHashDiff || isStatusDiff
+    return isHashDiff || isExpandedStatusDiff
   }
 
   componentWillMount () {
@@ -37,9 +36,25 @@ export default class NavigationTag extends Component {
   }
 
   render () {
-    const { title, status, methods, location } = this.props
+    const { title, shouldBeExpanded, methods, location } = this.props
+
+    // If tag has any method that matches location hash, then it is considered active
+    let isActiveTag = false
+    if (methods) {
+      isActiveTag = methods.some(method => (`#${method.link}` === location.hash))
+    }
 
-    const isExpanded = (status === 'right')
+    let isExpanded = false
+    if (shouldBeExpanded || isActiveTag) {
+      isExpanded = true
+    }
+
+    let indicatorDirection
+    if (isExpanded) {
+      indicatorDirection = 'bottom'
+    } else {
+      indicatorDirection = 'right'
+    }
 
     return (
       <div>
@@ -49,13 +64,13 @@ export default class NavigationTag extends Component {
           onClick={this.handleClick}
         >
           {title}
-          <Indicator status={status} />
+          <Indicator direction={indicatorDirection} />
         </a>
         <div className='nav-tag-methods'>
-          {isExpanded && methods && methods.map((method) => {
+          {methods && methods.map((method) => {
             const isActive = (`#${method.link}` === location.hash)
 
-            return <NavigationMethod key={method.link} method={method} isActive={isActive} />
+            return <NavigationMethod key={method.link} method={method} isActive={isActive} isOpen={isExpanded} />
           })}
         </div>
       </div>
@@ -66,7 +81,7 @@ export default class NavigationTag extends Component {
 NavigationTag.propTypes = {
   title: PropTypes.string.isRequired,
   methods: PropTypes.array,
-  status: PropTypes.string,
+  shouldBeExpanded: PropTypes.bool,
   onClick: PropTypes.func.isRequired,
   location: PropTypes.object
 }
diff --git a/src/components/NavigationTag/NavigationTag.scss b/src/components/NavigationTag/NavigationTag.scss
index 052d857..1c2a636 100644
--- a/src/components/NavigationTag/NavigationTag.scss
+++ b/src/components/NavigationTag/NavigationTag.scss
@@ -9,4 +9,12 @@
   &.active {
     background-color: $grey-dark;
   }
+
+  &.is-open {
+    display: block;
+  }
+
+  &.is-closed {
+    display: none;
+  }
 }
diff --git a/src/components/ServiceContainer/ServiceContainer.js b/src/components/ServiceContainer/ServiceContainer.js
index 25e7e61..430e757 100644
--- a/src/components/ServiceContainer/ServiceContainer.js
+++ b/src/components/ServiceContainer/ServiceContainer.js
@@ -1,25 +1,19 @@
 import React, { PureComponent } from 'react'
-import ScrollableAnchor from 'react-scrollable-anchor'
 import PropTypes from 'prop-types'
-
 import Method from '../Method/Method'
 
-// import './ServiceContainer.scss';
-
 export default class ServiceContainer extends PureComponent {
   render () {
     const { service } = this.props
     const { title, methods } = service
 
     return (
-      <ScrollableAnchor id={title}>
-        <div className='service-container'>
-          <h2>{title}</h2>
-          {methods.map(
-            (method) => <Method key={method.link} method={method} />
-          )}
-        </div>
-      </ScrollableAnchor>
+      <div className='service-container' id={title}>
+        <h2>{title}</h2>
+        {methods.map(
+          (method) => <Method key={method.link} method={method} />
+        )}
+      </div>
     )
   }
 }
diff --git a/src/handlers/BaseHandler.js b/src/handlers/BaseHandler.js
index 490710a..ebd7a64 100644
--- a/src/handlers/BaseHandler.js
+++ b/src/handlers/BaseHandler.js
@@ -15,7 +15,7 @@ class BaseHandler extends Component {
     const openApiUrl = this.props.location.query.url
     const parserType = this.props.parserType
     this.props.getDefinition(openApiUrl, parserType)
-    configureAnchors({ offset: 10 })
+    configureAnchors({ offset: -10, scrollDuration: 100 })
   }
 
   render () {