moved scrollTo to singleFeedContainer

Author: sumitdaga

src/projects/detail/containers/FeedContainer.js

@@ -32,7 +32,6 @@ import Section from '../components/Section'
 import SectionTitle from '../components/SectionTitle'
 import SingleFeedContainer from './SingleFeedContainer'
 
-import { scrollToHash } from '../../../components/ScrollToAnchors'
 import { isSystemUser } from '../../../helpers/tcHelpers'
 import { checkPermission } from '../../../helpers/permissions'
 import PERMISSIONS from '../../../config/permissions'
@@ -216,15 +215,6 @@ class FeedView extends React.Component {
         const resetNewComment = prevFeed && prevFeed.isAddingComment && !feed.isAddingComment && !feed.error
         return this.mapFeed(feed, this.state.showAll.indexOf(feed.id) > -1, resetNewComment, prevProps)
       }).filter(item => item)
-    }, () => {
-      if (prevProps) {
-        // only scroll at first time
-        return
-      }
-      const scrollTo = window.location.hash ? window.location.hash.substring(1) : null
-      if (scrollTo) {
-        setTimeout(() => scrollToHash(scrollTo), 100)
-      }
     })
   }
 

src/projects/detail/containers/SingleFeedContainer.jsx

@@ -14,6 +14,7 @@ import React from 'react'
 import _ from 'lodash'
 
 import ScrollableFeed from '../../../components/Feed/ScrollableFeed'
+import { scrollToHash } from '../../../components/ScrollToAnchors'
 
 const bindMethods = [
   'onNewCommentChange',
@@ -43,6 +44,14 @@ class SingleFeedContainer extends React.Component {
     })
   }
 
+  componentDidMount() {
+    // we use this to just scroll to a feed block or comment in a feed block, if there is a url hash
+    const scrollTo = window.location.hash ? window.location.hash.substring(1) : null
+    if (scrollTo) {
+      setTimeout(() => scrollToHash(scrollTo), 100)
+    }
+  }
+
   render() {
     const nonBindProps = _.omit(this.props, bindMethods)
     const bindProps = _.zipObject(bindMethods, bindMethods.map((method) => this[method]))