#282 Covid.view.MainContainerController: onHashChange() => delay to apply the new country selection when switching cards, button changeRoute adjustments

Author: tobiu

apps/covid/view/MainContainerController.mjs

@@ -286,6 +286,7 @@ class MainContainerController extends ComponentController {
             activeIndex  = me.getTabIndex(value),
             tabContainer = me.getReference('tab-container'),
             activeView   = me.getView(activeIndex),
+            switchView   = tabContainer.activeIndex !== activeIndex,
             id;
 
         // console.log('onHashChange', value);
@@ -300,16 +301,18 @@ class MainContainerController extends ComponentController {
         }
 
         if (value.country) {
-            if (activeView.ntype === 'table-container') {
-                id = activeView.selectionModel.getRowId(activeView.store.indexOf(value.country));
-                activeView.selectionModel.select(id);
-
-                Neo.main.DomAccess.scrollIntoView({
-                    id: id
-                });
-            } else {
-                activeView.selectionModel.select(value.country);
-            }
+            setTimeout(() => {
+                if (activeView.ntype === 'table-container') {
+                    id = activeView.selectionModel.getRowId(activeView.store.indexOf(value.country));
+                    activeView.selectionModel.select(id);
+
+                    Neo.main.DomAccess.scrollIntoView({
+                        id: id
+                    });
+                } else {
+                    activeView.selectionModel.select(value.country);
+                }
+            }, switchView ? 100 : 0);
         }
     }
 

src/Main.mjs

@@ -128,6 +128,10 @@ class Main extends core.Base {
         let hashObj = DomEvents.parseHash(window.location.hash.substr(1)),
             hashArr = [];
 
+        if (typeof data === 'string') {
+            data = DomEvents.parseHash(data);
+        }
+
         Object.assign(hashObj, data);
 
         Object.entries(hashObj).forEach(([key, value]) => {

src/component/Button.mjs

@@ -275,9 +275,15 @@ class Button extends Component {
      * @private
      */
     changeRoute() {
-        Neo.Main[this.editRoute ? 'editRoute' : 'setRoute']({
-            value: this.route
-        });
+        const me = this;
+
+        if (me.editRoute) {
+            Neo.Main.editRoute(this.route);
+        } else {
+            Neo.Main.setRoute({
+                value: me.route
+            });
+        }
     }
 }