From 39d11063d686d61fc4cf33ff1a4b7b3234372228 Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Wed, 5 Jul 2023 12:59:23 +0200
Subject: [PATCH] Open default homepage directly on navbar icon click

---
 src/components/NavBar.vue       | 10 ++--------
 src/components/TrendingPage.vue | 12 +-----------
 src/main.js                     | 14 ++++++++++++++
 3 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/src/components/NavBar.vue b/src/components/NavBar.vue
index 8f4518c1..0915b6f8 100644
--- a/src/components/NavBar.vue
+++ b/src/components/NavBar.vue
@@ -1,14 +1,8 @@
 <template>
     <nav class="flex flex-wrap items-center justify-center px-2 sm:px-4 pb-2.5 w-full relative">
         <div class="flex-1 flex justify-start">
-            <router-link class="flex font-bold text-3xl items-center font-sans" to="/"
-                ><img
-                    alt="logo"
-                    src="/img/icons/logo.svg"
-                    height="32"
-                    width="32"
-                    class="w-10 mr-[-0.6rem]"
-                />iped</router-link
+            <span class="flex font-bold text-3xl items-center font-sans cursor-pointer" @click="openHomePage(this)"
+                ><img alt="logo" src="/img/icons/logo.svg" height="32" width="32" class="w-10 mr-[-0.6rem]" />iped</span
             >
         </div>
         <div class="lt-md:hidden search-container">
diff --git a/src/components/TrendingPage.vue b/src/components/TrendingPage.vue
index 34dfec23..d10d2cf1 100644
--- a/src/components/TrendingPage.vue
+++ b/src/components/TrendingPage.vue
@@ -34,17 +34,7 @@ export default {
     activated() {
         document.title = this.$t("titles.trending") + " - Piped";
         if (this.videos.length > 0) this.updateWatched(this.videos);
-        if (this.$route.path == "/") {
-            switch (this.getPreferenceString("homepage", "trending")) {
-                case "trending":
-                    break;
-                case "feed":
-                    this.$router.push("/feed");
-                    return;
-                default:
-                    break;
-            }
-        }
+        if (this.$route.path == "/") this.openHomePage(this);
     },
     methods: {
         async fetchTrending(region) {
diff --git a/src/main.js b/src/main.js
index 138e4602..e2e82d37 100644
--- a/src/main.js
+++ b/src/main.js
@@ -532,6 +532,20 @@ const mixin = {
                 },
             });
         },
+        openHomePage(_this) {
+            let route;
+            switch (_this.getPreferenceString("homepage", "trending")) {
+                case "trending":
+                    route = "/trending";
+                    break;
+                case "feed":
+                    route = "/feed";
+                    break;
+                default:
+                    break;
+            }
+            if (route !== undefined) _this.$router.push(route);
+        },
     },
     computed: {
         authenticated(_this) {