Merge pull request #565 from appirio-tech/feature/project_page_scroll_position

vikasrohit · web-flow · commit 9fdca35df8a7 · 2016-11-29T12:21:55.000+05:30
Github issue #552, Project listing: return user to previous spot in listing after back from project
diff --git a/src/projects/list/components/Projects/Projects.jsx b/src/projects/list/components/Projects/Projects.jsx
@@ -15,7 +15,7 @@ class Projects extends Component {
   }
 
   componentDidUpdate() {
-      window.scrollTo(0, parseInt(window.sessionStorage.getItem("projectsPageScrollTop")));
+    window.scrollTo(0, parseInt(window.sessionStorage.getItem("projectsPageScrollTop")));
   }
 
   componentWillUnmount(){
@@ -45,6 +45,7 @@ class Projects extends Component {
   }
 
   onPageChange(pageNum) {
+    window.sessionStorage.removeItem('projectsPageScrollTop');
     this.routeWithParams(this.props.criteria, pageNum)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ class Projects extends Component {`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`componentDidUpdate() {`
`18`		`- window.scrollTo(0, parseInt(window.sessionStorage.getItem("projectsPageScrollTop")));`
	`18`	`+ window.scrollTo(0, parseInt(window.sessionStorage.getItem("projectsPageScrollTop")));`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`componentWillUnmount(){`
`@@ -45,6 +45,7 @@ class Projects extends Component {`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`onPageChange(pageNum) {`
	`48`	`+ window.sessionStorage.removeItem('projectsPageScrollTop');`
`48`	`49`	`this.routeWithParams(this.props.criteria, pageNum)`
`49`	`50`	`}`
`50`	`51`