Merge pull request #1191 from opentripplanner/itineraryview-url-param…

…-mobile-only

Avoid setting ui_itineraryView on desktop

lib/actions/ui.js

@@ -375,15 +375,20 @@ export function handleBackButtonPress(e) {
  */
 export function setItineraryView(value) {
   return function (dispatch, getState) {
-    const urlParams = coreUtils.query.getUrlParams()
+    if (coreUtils.ui.isMobile()) {
+      const urlParams = coreUtils.query.getUrlParams()
 
-    // If the itinerary value is changed,
-    // set the desired ui query param (even if LIST, so it replaces the current value)
-    // and store the current view as previousItineraryView.
-    if (value !== getItineraryView(urlParams)) {
-      urlParams.ui_itineraryView = value
+      // If the itinerary value is changed,
+      // set the desired ui query param (even if LIST, so it replaces the current value)
+      // and store the current view as previousItineraryView.
+      if (value !== getItineraryView(urlParams)) {
+        urlParams.ui_itineraryView = value
 
-      dispatch(setUrlSearch(urlParams))
+        dispatch(setUrlSearch(urlParams))
+        dispatch(settingItineraryView(value))
+      }
+    } else {
+      // Don't set/use the ui_itineraryView in desktop mode.
       dispatch(settingItineraryView(value))
     }
   }