From f9c27327364854a12f06e240d67f215a6cd8da6a Mon Sep 17 00:00:00 2001
From: MD1125 <166238654+MD1125@users.noreply.github.com>
Date: Tue, 10 Jun 2025 20:20:50 +0200
Subject: [PATCH] Unified theme for all Sign in buttons
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Um, so I made all the Sign in buttons look the same now - same like styling and functiounality across all of them.
Also, the button that used to send people to /login (which doesn’t even exist .-. ) now opens the sign-in popup instead.
Did the same for the one on the coinSymbol page so no more redirect to "/", it just opens the popup now.
Oh, and the comments section button also opens the Popup. Just annoyed me that they looked differently and redirected to 404 Pages.
---
 .../lib/components/self/CommentSection.svelte |  2 +-
 .../src/routes/coin/[coinSymbol]/+page.svelte |  6 +++++-
 website/src/routes/gambling/+page.svelte      |  7 +------
 website/src/routes/portfolio/+page.svelte     | 19 ++++++++++++-------
 website/src/routes/settings/+page.svelte      | 19 ++++++++++++++++---
 5 files changed, 35 insertions(+), 18 deletions(-)

diff --git a/website/src/lib/components/self/CommentSection.svelte b/website/src/lib/components/self/CommentSection.svelte
index d7af7d6..c49d6dd 100644
--- a/website/src/lib/components/self/CommentSection.svelte
+++ b/website/src/lib/components/self/CommentSection.svelte
@@ -219,7 +219,7 @@
 		{:else}
 			<div class="text-center">
 				<p class="text-muted-foreground mb-3 text-sm">Sign in to join the discussion</p>
-				<Button onclick={() => goto('/')} size="sm">Sign In</Button>
+				<Button onclick={() => (shouldSignIn = true)} size="sm">Sign In</Button>
 			</div>
 		{/if}
 
diff --git a/website/src/routes/coin/[coinSymbol]/+page.svelte b/website/src/routes/coin/[coinSymbol]/+page.svelte
index 3eb3d85..3a36913 100644
--- a/website/src/routes/coin/[coinSymbol]/+page.svelte
+++ b/website/src/routes/coin/[coinSymbol]/+page.svelte
@@ -27,6 +27,7 @@
 	import { getPublicUrl, getTimeframeInSeconds, timeToLocal } from '$lib/utils.js';
 	import { websocketController, type PriceUpdate, isConnectedStore } from '$lib/stores/websocket';
 	import SEO from '$lib/components/self/SEO.svelte';
+	import SignInConfirmDialog from '$lib/components/self/SignInConfirmDialog.svelte';
 
 	const { data } = $props();
 	let coinSymbol = $derived(data.coinSymbol);
@@ -39,6 +40,7 @@
 	let sellModalOpen = $state(false);
 	let selectedTimeframe = $state('1m');
 	let lastPriceUpdateTime = 0;
+	let shouldSignIn = $state(false);
 
 	let previousCoinSymbol = $state<string | null>(null);
 
@@ -370,6 +372,8 @@
 	imageAlt={coin ? `${coin.name} (${coin.symbol}) logo` : `${coinSymbol} cryptocurrency logo`}
 />
 
+<SignInConfirmDialog bind:open={shouldSignIn} />
+
 {#if coin}
 	<TradeModal bind:open={buyModalOpen} type="BUY" {coin} onSuccess={handleTradeSuccess} />
 	<TradeModal
@@ -552,7 +556,7 @@
 							{:else}
 								<div class="py-4 text-center">
 									<p class="text-muted-foreground mb-3 text-sm">Sign in to start trading</p>
-									<Button onclick={() => goto('/')}>Sign In</Button>
+									<Button onclick={() => (shouldSignIn = true)}>Sign In</Button>
 								</div>
 							{/if}
 						</Card.Content>
diff --git a/website/src/routes/gambling/+page.svelte b/website/src/routes/gambling/+page.svelte
index 96183d1..5f00d20 100644
--- a/website/src/routes/gambling/+page.svelte
+++ b/website/src/routes/gambling/+page.svelte
@@ -55,12 +55,7 @@
 			<div class="text-center">
 				<div class="text-muted-foreground mb-4 text-xl">Sign in to start gambling</div>
 				<p class="text-muted-foreground mb-4 text-sm">You need an account to place bets</p>
-				<button
-					class="text-primary underline hover:cursor-pointer"
-					onclick={() => (shouldSignIn = true)}
-				>
-					Sign in to continue
-				</button>
+				<Button onclick={() => (shouldSignIn = true)}>Sign In</Button>
 			</div>
 		</div>
 	{:else}
diff --git a/website/src/routes/portfolio/+page.svelte b/website/src/routes/portfolio/+page.svelte
index 56a4d4d..a568344 100644
--- a/website/src/routes/portfolio/+page.svelte
+++ b/website/src/routes/portfolio/+page.svelte
@@ -11,6 +11,7 @@
 	import { goto } from '$app/navigation';
 	import { USER_DATA } from '$lib/stores/user-data';
 	import SendMoneyModal from '$lib/components/self/SendMoneyModal.svelte';
+	import SignInConfirmDialog from '$lib/components/self/SignInConfirmDialog.svelte';
 
 	// TODO: add type definitions
 	let portfolioData = $state<any>(null);
@@ -18,6 +19,7 @@
 	let loading = $state(true);
 	let error = $state<string | null>(null);
 	let sendMoneyModalOpen = $state(false);
+	let shouldSignIn = $state(false);
 
 	onMount(async () => {
 		await Promise.all([fetchPortfolioData(), fetchRecentTransactions()]);
@@ -234,6 +236,7 @@
 />
 
 <SendMoneyModal bind:open={sendMoneyModalOpen} onSuccess={handleTransferSuccess} />
+<SignInConfirmDialog bind:open={shouldSignIn} />
 
 <div class="container mx-auto max-w-7xl p-6">
 	<div class="mb-6 flex flex-col items-start justify-between gap-4 sm:flex-row sm:items-center">
@@ -241,12 +244,14 @@
 			<h1 class="text-3xl font-bold">Portfolio</h1>
 			<p class="text-muted-foreground">Manage your investments and transactions</p>
 		</div>
-		<div class="flex gap-2">
-			<Button onclick={() => (sendMoneyModalOpen = true)}>
-				<Send class="h-4 w-4" />
-				Send Money
-			</Button>
-		</div>
+		{#if $USER_DATA}
+			<div class="flex gap-2">
+				<Button onclick={() => (sendMoneyModalOpen = true)}>
+					<Send class="h-4 w-4" />
+					Send Money
+				</Button>
+			</div>
+		{/if}
 	</div>
 
 	{#if loading}
@@ -257,7 +262,7 @@
 				<div class="text-muted-foreground mb-4 text-xl">
 					You need to be logged in to view your portfolio
 				</div>
-				<Button onclick={() => goto('/login')}>Log In</Button>
+				<Button onclick={() => (shouldSignIn = true)}>Sign In</Button>
 			</div>
 		</div>
 	{:else if error}
diff --git a/website/src/routes/settings/+page.svelte b/website/src/routes/settings/+page.svelte
index 5750af0..d60e9db 100644
--- a/website/src/routes/settings/+page.svelte
+++ b/website/src/routes/settings/+page.svelte
@@ -16,7 +16,9 @@
 	import { USER_DATA } from '$lib/stores/user-data';
 	import * as Dialog from '$lib/components/ui/dialog';
 	import SEO from '$lib/components/self/SEO.svelte';
+	import SignInConfirmDialog from '$lib/components/self/SignInConfirmDialog.svelte';
 
+	let shouldSignIn = $state(false);
 	let name = $state($USER_DATA?.name || '');
 	let bio = $state($USER_DATA?.bio ?? '');
 	let username = $state($USER_DATA?.username || '');
@@ -36,7 +38,7 @@
 			avatarFile !== undefined
 	);
 
-	let fileInput: HTMLInputElement;
+	let fileInput: HTMLInputElement | undefined = $state(undefined);
 
 	let loading = $state(false);
 	let usernameAvailable: boolean | null = $state(null);
@@ -66,7 +68,7 @@
 	});
 
 	function handleAvatarClick() {
-		fileInput.click();
+		fileInput?.click();
 	}
 	function handleAvatarChange(e: Event) {
 		const f = (e.target as HTMLInputElement).files?.[0];
@@ -293,6 +295,16 @@
 <div class="container mx-auto max-w-2xl p-6">
 	<h1 class="mb-6 text-2xl font-bold">Settings</h1>
 
+	{#if !$USER_DATA}
+	<div class="flex h-96 items-center justify-center">
+		<div class="text-center">
+			<div class="text-muted-foreground mb-4 text-xl">
+				You need to be logged in to view your settings
+			</div>
+			<Button onclick={() => (shouldSignIn = true)}>Sign In</Button>
+		</div>
+	</div>
+{:else}
 	<div class="space-y-6">
 		<Card.Root>
 			<Card.Header>
@@ -476,6 +488,7 @@
 			</Card.Content>
 		</Card.Root>
 	</div>
+	{/if}
 </div>
 
 <Dialog.Root bind:open={deleteDialogOpen}>
@@ -519,4 +532,4 @@
 			</Button>
 		</Dialog.Footer>
 	</Dialog.Content>
-</Dialog.Root>
+</Dialog.Root>
\ No newline at end of file