From 4d3741c3b452b9fe57b8a44bd9f33eef28447ac5 Mon Sep 17 00:00:00 2001
From: Connor Turland <connorturland@gmail.com>
Date: Wed, 15 Mar 2017 22:02:58 -0400
Subject: [PATCH] handle page title and mobile title updates

---
 app/views/explore/mine.html.erb            |  2 +-
 app/views/explore/shared.html.erb          |  2 +-
 frontend/src/Metamaps/GlobalUI/ReactApp.js |  1 -
 frontend/src/Metamaps/Views/ExploreMaps.js | 36 ++++++++++++++++++----
 frontend/src/components/App/index.js       |  5 ++-
 5 files changed, 34 insertions(+), 12 deletions(-)

diff --git a/app/views/explore/mine.html.erb b/app/views/explore/mine.html.erb
index 21f034c3..21295d4f 100644
--- a/app/views/explore/mine.html.erb
+++ b/app/views/explore/mine.html.erb
@@ -5,7 +5,7 @@
    # %>
 
 <script>
-  <% content_for :title, "Explore My Maps | Metamaps" %>
+  <% content_for :title, "My Maps | Metamaps" %>
   <% content_for :mobile_title, "My Maps" %>
 
   Metamaps.currentPage = "mine";
diff --git a/app/views/explore/shared.html.erb b/app/views/explore/shared.html.erb
index 0f01fcf0..f40e9fdf 100644
--- a/app/views/explore/shared.html.erb
+++ b/app/views/explore/shared.html.erb
@@ -5,7 +5,7 @@
    # %>
 
 <script>
-  <% content_for :title, "Explore Shared Maps | Metamaps" %>
+  <% content_for :title, "Shared Maps | Metamaps" %>
   <% content_for :mobile_title, "Shared With Me" %>
 
   Metamaps.currentPage = "shared";
diff --git a/frontend/src/Metamaps/GlobalUI/ReactApp.js b/frontend/src/Metamaps/GlobalUI/ReactApp.js
index 22ee02ff..f4caa655 100644
--- a/frontend/src/Metamaps/GlobalUI/ReactApp.js
+++ b/frontend/src/Metamaps/GlobalUI/ReactApp.js
@@ -46,7 +46,6 @@ const ReactApp = {
   handleUpdate: function(location) {
     const self = ReactApp
     const pathname = this.state.location.pathname
-    // TODO: also handle page title updates
     switch (pathname.split('/')[1]) {
       case '':
         if (Active.Mapper && Active.Mapper.id) {
diff --git a/frontend/src/Metamaps/Views/ExploreMaps.js b/frontend/src/Metamaps/Views/ExploreMaps.js
index 506fb041..874ca1e7 100644
--- a/frontend/src/Metamaps/Views/ExploreMaps.js
+++ b/frontend/src/Metamaps/Views/ExploreMaps.js
@@ -16,15 +16,37 @@ const ExploreMaps = {
   mapper: null,
   updateFromPath: function(path) {
     const self = ExploreMaps
-    const test = path.split('/')[1]
-    const section = path.split('/')[2]
-    const id = path.split('/')[3]
+    const [_unused, generalSection, specificSection, id] = path.split('/')
 
-    if (test === 'explore') {
-      const capitalize = section.charAt(0).toUpperCase() + section.slice(1)
+    if (generalSection === 'explore') {
+      const capitalize = specificSection.charAt(0).toUpperCase() + specificSection.slice(1)
       self.setCollection(DataModel.Maps[capitalize])
-    } else if (test === '') {
+      switch (capitalize) {
+        case 'Active':
+          document.title = 'Explore Active Maps | Metamaps'
+          ReactApp.mobileTitle='Recently Active'
+          break
+        case 'Featured':
+          document.title = 'Explore Featured Maps | Metamaps'
+          ReactApp.mobileTitle='Featured Maps'
+          break
+        case 'Starred':
+          document.title = 'Starred Maps | Metamaps'
+          ReactApp.mobileTitle='Starred Maps'
+          break
+        case 'Shared':
+          document.title = 'Shared Maps | Metamaps'
+          ReactApp.mobileTitle='Shared With Me'
+          break
+        case 'Mine':
+          document.title = 'My Maps | Metamaps'
+          ReactApp.mobileTitle = 'My Maps'
+          break
+      }
+    } else if (generalSection === '') {
       self.setCollection(DataModel.Maps.Active)
+      document.title = 'Explore Active Maps | Metamaps'
+      ReactApp.mobileTitle='Recently Active'
     }
 
     if (id) {
@@ -98,6 +120,8 @@ const ExploreMaps = {
       url: '/users/' + self.collection.mapperId + '/details.json',
       success: function(response) {
         self.mapper = response
+        document.title = self.mapper.name + ' | Metamaps'
+        ReactApp.mobileTitle = self.mapper.name
         self.render()
       },
       error: function() {
diff --git a/frontend/src/components/App/index.js b/frontend/src/components/App/index.js
index 48364216..ddf9f16c 100644
--- a/frontend/src/components/App/index.js
+++ b/frontend/src/components/App/index.js
@@ -25,13 +25,12 @@ class App extends Component {
   }
 
   static childContextTypes = {
-    currentUser: PropTypes.object,
     location: PropTypes.object
   }
 
   getChildContext () {
-    const { route, location } = this.props
-    return {currentUser: route.currentUser, location}
+    const { location } = this.props
+    return {location}
   }
 
   render () {