:recycle: adding sort namespace to routes

vue-theme/src/components/sidebars/sidebar.vue

@@ -6,7 +6,7 @@ aside.sidebar
             h3.t-up sort {{ type }} by
             ul.t-up
                 li(v-for="option in sortOptions")
-                    router-link(:to="`/${type}/${sortTypes[option]}`")
+                    router-link(:to="`/${type}/sorted/${sortTypes[option]}`")
                         p {{ option }}
 
         
@@ -19,7 +19,7 @@ aside.sidebar
             h3.t-up sort {{ type }} by
             ul.t-up
                 li(v-for="option in sortOptions")
-                    router-link(:to="`/${type}/${sortTypes[option]}`")
+                    router-link(:to="`/${type}/sorted/${sortTypes[option]}`")
                         p {{ option }}
                 //- temporary reminder
                 p 
@@ -31,7 +31,7 @@ aside.sidebar
             h3.t-up sort {{ type }} by
             ul.t-up
                 li(v-for="option in sortOptions")
-                    router-link(:to="`/${type}/${sortTypes[option]}`")
+                    router-link(:to="`/${type}/sorted/${sortTypes[option]}`")
                         p {{ option }}
                 //- temporary reminder
                 p 

vue-theme/src/pages/list.vue

@@ -6,7 +6,7 @@
             .title.f-row
                 h3 {{ type }} list
                 span(v-if="sortBy")
-                    h3 sorted {{ sortBy.replace('-', ' ') }}
+                    h3  sorted {{ sortBy.replace('-', ' ') }}
 
             h3(v-if="!loaded") loading...
             .content(
@@ -38,14 +38,12 @@ import { postTypes, sortTypes, convertTitleCase, typeFromRoute, sortFromRoute }
 const TIMEOUT = 1
 const INTERSECT_SELECTOR = '.page--list > article footer'
 
+const wideTypes = ['event', 'exhibition']
+const gridTypes = ['episode', 'artist']
+const sansSidebarTypes = ['episode']
+
 export default {
     components: { sidebar, featuredImage, card },
-    props: {
-        sidebar: { type: Boolean },
-        grid: { type: Boolean },
-        sortBy: { type: String },
-        isWide: { type: Boolean }
-    },
     mixins: [postTypeGetters, scrollTop, heroUtils],
     data() {
         return {
@@ -58,9 +56,21 @@ export default {
         }
     },
     computed: {
+        grid() {
+            return gridTypes.includes(this.type)
+        },
+        isWide() {
+            return wideTypes.includes(this.type)
+        },
+        sidebar() {
+            return !sansSidebarTypes.includes(this.type)
+        },
         type() {
             // Checks for type and fixes Episodes route edge case
-            return typeFromRoute(this.$route)
+            return this.$route.params.type
+        },
+        sortBy() {
+            return this.$route.params.sortBy
         },
         pType() {
             if(!typeFromRoute(this.$route)) return
@@ -135,20 +145,24 @@ export default {
         },
         async getPageForType(type) {
             await this._getAll('page', this.$store)
-            if(!this.allPages) return console.warn('no pages in state', this)
+            if(!this.allPages) throw 'no pages in state'
             const page = this.allPages.filter(page => page.slug == `${type}s`)[0]
-            if(!page) return console.warn(`no page for ${type} found`)
+            if(!page) throw `No page: ${type} found. Cannot set hero.`
             return page
         },
         // _setHeroInfo(post) {} from mixin
         // _clearHero(store) {} from mixin
         async checkAndSetHero(type) {
             this._clearHero(this.$store)
-            const page = await this.getPageForType(type)
-            // We always set a hero no matter what
-            // Because the hero component will deal
-            // with how to render based on hero.url
-            this.$store.commit('SET_HERO', this._setHeroInfo(page))
+            try {
+                const page = await this.getPageForType(type)
+                // We always set a hero no matter what
+                // Because the hero component will deal
+                // with how to render based on hero.url
+                this.$store.commit('SET_HERO', this._setHeroInfo(page))
+            } catch (err) {
+                console.error(err)
+            }
         },
         setIntersectionLoader() {
             // KeepFetching is UNSET for certain post types and sort types in `loadMorePosts()`
@@ -221,6 +235,9 @@ export default {
     },
     mounted() {
         // This only fires navigating from a non-list page > list page
+        console.log(this.$route)
+        console.log(this.type)
+        console.log(this.sortBy)
         this.clearAndInitPostList('mounted')
     },
     beforeDestroy() {

vue-theme/src/router/routes.js

@@ -6,7 +6,7 @@ import { sortTypes } from '@/utils/helpers'
 
 const gridWithSidebar = { sidebar: true, grid: true }
 const gridSansSidebar = { sidebar: false, grid: true }
-const sansGridWithSidebar = { sidebar: true, grid: false }
+const sansGridWithSidebar = { sidebar: true, grid: false, isWide: true }
 
 export default [
     // Home Page
@@ -14,16 +14,17 @@ export default [
     // List Pages
     {
         path: '/:type',
-        component: listPage
+        component: listPage,
+        props: gridWithSidebar,
     },
     {
         path: '/:type/sorted/:sortBy',
-        component: listPage
+        component: listPage,
+        props: gridWithSidebar,
     },
     // Single Pages
     {
         path: '/:type/:slug',
         component: singlePage,
-        props: { sidebar: true },
     },
-]
+]