From b01650c7c14c13e87e6c176640ce0e8250e4e711 Mon Sep 17 00:00:00 2001
From: naskya <m@naskya.net>
Date: Thu, 20 Jul 2023 06:56:00 +0000
Subject: [PATCH] feat: prioritize favicons in instance tickers (close #32)

---
 README.md                                           | 2 ++
 packages/client/src/components/MkInstanceTicker.vue | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index c140c35a..fb809b7b 100644
--- a/README.md
+++ b/README.md
@@ -13,6 +13,8 @@
 
 ## 細かい変更点
 
+- インスタンスティッカーに表示するサーバーのアイコンとして favicon を優先する
+  - favicon のほうがよくカスタマイズされているため
 - 誤爆しやすい位置にあるフォローボタンを隠す設定を追加
   - フォローを誤爆すると悲しいため
 - クローラーを拒否する [robots.txt](./custom/assets/robots.txt) の追加
diff --git a/packages/client/src/components/MkInstanceTicker.vue b/packages/client/src/components/MkInstanceTicker.vue
index d998ca2f..9e14e3d8 100644
--- a/packages/client/src/components/MkInstanceTicker.vue
+++ b/packages/client/src/components/MkInstanceTicker.vue
@@ -28,7 +28,7 @@ let ticker = $ref<HTMLElement | null>(null);
 
 // if no instance data is given, this is for the local instance
 const instance = props.instance ?? {
-	faviconUrl: Instance.iconUrl || Instance.faviconUrl || "/favicon.ico",
+	faviconUrl: Instance.faviconUrl || Instance.iconUrl || "/favicon.ico",
 	name: instanceName,
 	themeColor: (
 		document.querySelector(
@@ -50,8 +50,8 @@ const bg = {
 
 function getInstanceIcon(instance): string {
 	return (
-		getProxiedImageUrlNullable(instance.iconUrl, "preview") ??
 		getProxiedImageUrlNullable(instance.faviconUrl, "preview") ??
+		getProxiedImageUrlNullable(instance.iconUrl, "preview") ??
 		"/client-assets/dummy.png"
 	);
 }