From cee01c9a298db3ecbf2733ccbeb6adad683f5759 Mon Sep 17 00:00:00 2001
From: Tim Wundenberg <tim@wundenbergs.de>
Date: Sat, 3 Jan 2026 16:33:56 +0100
Subject: [PATCH] feat(budget): further implementation with modal for editing

---
 internal/budget/handler.go     | 43 ++++++++++++++++++----
 internal/budget/template.templ | 66 ++++++++++++++++++++++++++--------
 internal/core/layout.templ     |  1 +
 static/js/layout.js            | 20 +++++++++--
 static/js/time.js              |  2 +-
 5 files changed, 106 insertions(+), 26 deletions(-)

diff --git a/internal/budget/handler.go b/internal/budget/handler.go
index c5efedd..d81ac9e 100644
--- a/internal/budget/handler.go
+++ b/internal/budget/handler.go
@@ -33,7 +33,7 @@ func NewHandler(s Service, r *core.Render) Handler {
 
 func (h HandlerImpl) Handle(r *http.ServeMux) {
 	r.Handle("GET /budget", h.handlePage())
-	// r.Handle("GET /budget/{id}", h.handleTransactionItemComp())
+	r.Handle("GET /budget/{id}", h.handleEdit())
 	r.Handle("POST /budget/{id}", h.handlePost())
 	// r.Handle("DELETE /budget/{id}", h.handleDelete())
 }
@@ -48,17 +48,46 @@ func (h HandlerImpl) handlePage() http.HandlerFunc {
 			return
 		}
 
-		// transactions, err := h.s.GetAll(r.Context(), user)
-		// if err != nil {
-		// 	core.HandleError(w, r, err)
-		// 	return
-		// }
+		budgets, err := h.s.GetAll(r.Context(), user)
+		if err != nil {
+			core.HandleError(w, r, err)
+			return
+		}
 
-		comp := page()
+		comp := page(budgets)
 		h.r.RenderLayout(r, w, comp, user)
 	}
 }
 
+func (h HandlerImpl) handleEdit() http.HandlerFunc {
+	return func(w http.ResponseWriter, r *http.Request) {
+		core.UpdateSpan(r)
+
+		user := core.GetUser(r)
+		if user == nil {
+			core.DoRedirect(w, r, "/auth/signin")
+			return
+		}
+
+		var (
+		// id  uuid.UUID
+		// err error
+		)
+
+		idStr := r.PathValue("id")
+		if idStr != "new" {
+			// id, err = uuid.Parse(idStr)
+			// if err != nil {
+			// 	core.HandleError(w, r, fmt.Errorf("could not parse Id: %w", core.ErrBadRequest))
+			// 	return
+			// }
+		}
+
+		comp := editNew()
+		h.r.Render(r, w, comp)
+	}
+}
+
 func (h HandlerImpl) handlePost() http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		core.UpdateSpan(r)
diff --git a/internal/budget/template.templ b/internal/budget/template.templ
index c58ed21..89cb5d4 100644
--- a/internal/budget/template.templ
+++ b/internal/budget/template.templ
@@ -2,39 +2,75 @@ package budget
 
 import "spend-sparrow/internal/template/svg"
 
-templ page() {
+templ page(budgets []Budget) {
+	<!-- for _,name:=range([]string{"Lebensmittel", "Tanken", "Drogerie", "Milch", "Parken"}) { -->
 	<div class="flex flex-wrap gap-20 text-xl mt-10 justify-center">
 		@newItem()
-		for i:=range(10) {
-			@item(i)
+		for _,budget:=range(budgets ) {
+			@item(budget)
 		}
 	</div>
 }
 
-templ Items() {
-	<div></div>
-}
-
-templ Edit() {
-	<div></div>
+templ editNew() {
+	<dialog id="newBudget" class="absolute left-1/2 top-1/2 -translate-x-1/2 -translate-y-1/2 rounded-lg openDialogModal">
+		<form
+			hx-post={ "/transaction/new" }
+			class="flex justify-end gap-4 items-center "
+		>
+			<div class="grid grid-cols-[auto_auto] items-center gap-4 mr-auto">
+				<label for="timestamp" class="text-sm text-gray-500">Name</label>
+				<input
+					autofocus
+					name="name"
+					type="text"
+					class="bg-white input datetime"
+				/>
+				<label for="value" class="text-sm text-gray-500">Value</label>
+				<input
+					name="value"
+					type="number"
+					class="mr-auto bg-white input"
+				/>
+			</div>
+			<button type="submit" class="button button-neglect px-1 flex items-center gap-2">
+				@svg.Save()
+				<span>
+					Save
+				</span>
+			</button>
+			<button commandfor="newBudget" command="close" class="button button-neglect px-1 flex items-center gap-2">
+				<span class="h-4 w-4">
+					@svg.Cancel()
+				</span>
+				<span>
+					Cancel
+				</span>
+			</button>
+		</form>
+	</dialog>
 }
 
 templ newItem() {
-	<section class="p-5 flex gap-4 active:bg-gray-200 flex-col justify-center items-center hover:bg-gray-100 transition-all cursor-pointer w-44 h-44 rounded-lg bg-gray-50 border-1 border-gray-300">
+	<button
+		hx-get="/budget/new"
+		hx-target="#dialogContainer"
+		class="p-5 w-64 h-64 flex gap-10 active:bg-gray-200 flex-col justify-center items-center hover:bg-gray-100 transition-all cursor-pointer rounded-lg bg-gray-50 border-1 border-gray-300"
+	>
 		New Budget
 		<div class="w-10">
 			@svg.Plus()
 		</div>
-	</section>
+	</button>
 }
 
-templ item(i int) {
-	<section class="flex flex-col w-44 h-44 p-5 rounded-lg bg-gray-50 border-1 border-gray-300 ">
+templ item(budget Budget) {
+	<section class="flex flex-col w-64 h-64 p-5 rounded-lg bg-gray-50 border-1 border-gray-300 ">
 		<span>
-			Budget { i }
+			{ budget.Description }
 		</span>
 		<span>
-			{ 200 + i }€
+			{ 200 }€
 		</span>
 	</section>
 }
diff --git a/internal/core/layout.templ b/internal/core/layout.templ
index 68085db..348274f 100644
--- a/internal/core/layout.templ
+++ b/internal/core/layout.templ
@@ -37,6 +37,7 @@ templ Layout(slot templ.Component, user templ.Component, loggedIn bool, path str
 			<script src="/static/js/dashboard.js" defer></script>
 		</head>
 		<body hx-headers='{"Csrf-Token": "CSRF_TOKEN"}'>
+			<div id="dialogContainer"></div>
 			<div class="flex flex-col min-h-screen">
 				<header class="sticky top-0 z-50 bg-white flex items-center gap-6 p-4 border-b-1 border-gray-200">
 					<button id="menuButton" class="w-10 h-10 block xl:hidden">
diff --git a/static/js/layout.js b/static/js/layout.js
index afa6118..a66c30e 100644
--- a/static/js/layout.js
+++ b/static/js/layout.js
@@ -1,11 +1,25 @@
 
 document.addEventListener("DOMContentLoaded", () => {
-	menuButton.addEventListener("click", function (e) {
+	menuButton.addEventListener("click", function() {
 		menu.showModal();
 	});
-	menuButtonClose.addEventListener("click", function (e) {
+	menuButtonClose.addEventListener("click", function() {
 		menu.close();
 	});
-
 })
 
+htmx.on("htmx:afterSwap", () => {
+	dialogs = dialogContainer.getElementsByClassName("openDialogModal");
+	Array.from(dialogs).forEach((dialog) => {
+		dialog.showModal()
+		// buttons = dialog.getElementsByClassName("closeModalDialog");
+		// Array.from(buttons).forEach((button) => button.addEventListener("click", closeModalDialog))
+	})
+});
+
+// function closeModalDialog(e) {
+// 	e.preventDefault();
+// 	console.log(e)
+// 	e.target.close()
+// }
+
diff --git a/static/js/time.js b/static/js/time.js
index d4f41c7..af4aeb1 100644
--- a/static/js/time.js
+++ b/static/js/time.js
@@ -11,7 +11,7 @@ function updateTime() {
 	document.querySelectorAll(".datetime").forEach((el) => {
 		if (el.textContent !== "") {
 			el.textContent = el.textContent.includes("UTC") ? new Date(el.textContent).toLocaleString([], { day: 'numeric', month: 'short', year: 'numeric' }) : el.textContent;
-		} else if (el.attributes['value'] !== "") {
+		} else if (el.attributes['value'] && el.attributes['value'] !== "") {
 			const value = el.attributes['value'].value;
 			const newDate = value.includes("UTC") ? new Date(value) : value;
 			el.valueAsDate = newDate;