From 8a6380689a9bc5bffe5ed2027a3bff16beae572a Mon Sep 17 00:00:00 2001
From: Anbraten <anton@ju60.de>
Date: Wed, 9 Jul 2025 08:19:53 +0200
Subject: [PATCH 01/11] feat: partially load notifications list

---
 .../user/notification/notification_div.tmpl   | 37 ++++++++++---------
 web_src/js/features/notification.ts           |  1 +
 2 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index b8655a84a4e73..211306b0a7df9 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -73,12 +73,13 @@
 										{{$.CsrfTokenHtml}}
 										<input type="hidden" name="notification_id" value="{{.ID}}">
 										<input type="hidden" name="status" value="pinned">
-										<button class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
-											data-url="{{AppSubUrl}}/notifications/status"
-											data-status="pinned"
-											data-page="{{$.Page.Paginater.Current}}"
-											data-notification-id="{{.ID}}"
-											data-q="{{$.Keyword}}">
+										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
+											hx-post="{{AppSubUrl}}/notifications/status"
+											hx-target="#notification_div"
+											hx-swap="outerHTML"
+											hx-vals='{"notification_id": "{{.ID}}", "status": "pinned", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
+											{{if eq .Status 3}}disabled{{end}}
+											>
 											{{svg "octicon-pin"}}
 										</button>
 									</form>
@@ -89,12 +90,12 @@
 										<input type="hidden" name="notification_id" value="{{.ID}}">
 										<input type="hidden" name="status" value="read">
 										<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
-										<button class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
-											data-url="{{AppSubUrl}}/notifications/status"
-											data-status="read"
-											data-page="{{$.Page.Paginater.Current}}"
-											data-notification-id="{{.ID}}"
-											data-q="{{$.Keyword}}">
+										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
+											hx-post="{{AppSubUrl}}/notifications/status"
+											hx-target="#notification_div"
+											hx-swap="outerHTML"
+											hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
+											>
 											{{svg "octicon-check"}}
 										</button>
 									</form>
@@ -104,12 +105,12 @@
 										<input type="hidden" name="notification_id" value="{{.ID}}">
 										<input type="hidden" name="status" value="unread">
 										<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
-										<button class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
-											data-url="{{AppSubUrl}}/notifications/status"
-											data-status="unread"
-											data-page="{{$.Page.Paginater.Current}}"
-											data-notification-id="{{.ID}}"
-											data-q="{{$.Keyword}}">
+										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
+											hx-post="{{AppSubUrl}}/notifications/status"
+											hx-target="#notification_div"
+											hx-swap="outerHTML"
+											hx-vals='{"notification_id": "{{.ID}}", "status": "unread", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
+											>
 											{{svg "octicon-bell"}}
 										</button>
 									</form>
diff --git a/web_src/js/features/notification.ts b/web_src/js/features/notification.ts
index dc0acb0244edd..0d10f1cd032ac 100644
--- a/web_src/js/features/notification.ts
+++ b/web_src/js/features/notification.ts
@@ -144,6 +144,7 @@ async function updateNotificationCountWithCallback(callback: (timeout: number, n
 }
 
 async function updateNotificationTable() {
+  return;
   const notificationDiv = document.querySelector('#notification_div');
   if (notificationDiv) {
     try {

From 65071d5c2b1962bee510fcadaa72f5d8d6245f6e Mon Sep 17 00:00:00 2001
From: Anton Bracke <anton.bracke@fastleansmart.com>
Date: Wed, 9 Jul 2025 14:06:09 +0200
Subject: [PATCH 02/11] fix

---
 routers/web/user/notification.go              |  6 ++
 .../user/notification/notification_div.tmpl   | 67 +++++++------------
 2 files changed, 31 insertions(+), 42 deletions(-)

diff --git a/routers/web/user/notification.go b/routers/web/user/notification.go
index 610a9b8076059..c131dacadba3a 100644
--- a/routers/web/user/notification.go
+++ b/routers/web/user/notification.go
@@ -180,6 +180,12 @@ func NotificationStatusPost(ctx *context.Context) {
 		ctx.Redirect(url, http.StatusSeeOther)
 	}
 
+	// convert the action to the displayed list
+	if statusStr == "unread" {
+		ctx.SetFormString("q", "read")
+	} else {
+		ctx.SetFormString("q", "unread")
+	}
 	getNotifications(ctx)
 	if ctx.Written() {
 		return
diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index 211306b0a7df9..7f1ee202c2872 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -69,51 +69,34 @@
 							</div>
 							<div class="notifications-buttons tw-items-center tw-justify-end tw-gap-1 tw-px-1">
 								{{if ne .Status 3}}
-									<form action="{{AppSubUrl}}/notifications/status" method="post">
-										{{$.CsrfTokenHtml}}
-										<input type="hidden" name="notification_id" value="{{.ID}}">
-										<input type="hidden" name="status" value="pinned">
-										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
-											hx-post="{{AppSubUrl}}/notifications/status"
-											hx-target="#notification_div"
-											hx-swap="outerHTML"
-											hx-vals='{"notification_id": "{{.ID}}", "status": "pinned", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
-											{{if eq .Status 3}}disabled{{end}}
-											>
-											{{svg "octicon-pin"}}
-										</button>
-									</form>
+									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
+										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-target="#notification_div"
+										hx-swap="outerHTML"
+										hx-vals='{"notification_id": "{{.ID}}", "status": "pinned", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true" }'
+										{{if eq .Status 3}}disabled{{end}}
+										>
+										{{svg "octicon-pin"}}
+									</button>
 								{{end}}
 								{{if or (eq .Status 1) (eq .Status 3)}}
-									<form action="{{AppSubUrl}}/notifications/status" method="post">
-										{{$.CsrfTokenHtml}}
-										<input type="hidden" name="notification_id" value="{{.ID}}">
-										<input type="hidden" name="status" value="read">
-										<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
-										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
-											hx-post="{{AppSubUrl}}/notifications/status"
-											hx-target="#notification_div"
-											hx-swap="outerHTML"
-											hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
-											>
-											{{svg "octicon-check"}}
-										</button>
-									</form>
+									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
+										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-target="#notification_div"
+										hx-swap="outerHTML"
+										hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
+										>
+										{{svg "octicon-check"}} 123
+									</button>
 								{{else if eq .Status 2}}
-									<form action="{{AppSubUrl}}/notifications/status" method="post">
-										{{$.CsrfTokenHtml}}
-										<input type="hidden" name="notification_id" value="{{.ID}}">
-										<input type="hidden" name="status" value="unread">
-										<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
-										<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
-											hx-post="{{AppSubUrl}}/notifications/status"
-											hx-target="#notification_div"
-											hx-swap="outerHTML"
-											hx-vals='{"notification_id": "{{.ID}}", "status": "unread", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}"}'
-											>
-											{{svg "octicon-bell"}}
-										</button>
-									</form>
+									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
+										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-target="#notification_div"
+										hx-swap="outerHTML"
+										hx-vals='{"notification_id": "{{.ID}}", "status": "unread", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
+										>
+										{{svg "octicon-bell"}}
+									</button>
 								{{end}}
 							</div>
 						</div>

From a9c914f8c9962f4dc37f23736ae03b2a3afb9b22 Mon Sep 17 00:00:00 2001
From: Anton Bracke <anton.bracke@fastleansmart.com>
Date: Wed, 9 Jul 2025 14:26:58 +0200
Subject: [PATCH 03/11] fix

---
 templates/user/notification/notification_div.tmpl | 2 +-
 web_src/js/features/notification.ts               | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index 7f1ee202c2872..5dcd90159bd1c 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -86,7 +86,7 @@
 										hx-swap="outerHTML"
 										hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
 										>
-										{{svg "octicon-check"}} 123
+										{{svg "octicon-check"}}
 									</button>
 								{{else if eq .Status 2}}
 									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
diff --git a/web_src/js/features/notification.ts b/web_src/js/features/notification.ts
index 0d10f1cd032ac..dc0acb0244edd 100644
--- a/web_src/js/features/notification.ts
+++ b/web_src/js/features/notification.ts
@@ -144,7 +144,6 @@ async function updateNotificationCountWithCallback(callback: (timeout: number, n
 }
 
 async function updateNotificationTable() {
-  return;
   const notificationDiv = document.querySelector('#notification_div');
   if (notificationDiv) {
     try {

From 2e4add4362bab48c0d0d87698bbb7e968613bc2c Mon Sep 17 00:00:00 2001
From: Anton Bracke <anton.bracke@fastleansmart.com>
Date: Wed, 9 Jul 2025 14:58:26 +0200
Subject: [PATCH 04/11] improve code

---
 routers/web/user/notification.go                  | 6 ------
 templates/user/notification/notification_div.tmpl | 6 +++---
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/routers/web/user/notification.go b/routers/web/user/notification.go
index c131dacadba3a..610a9b8076059 100644
--- a/routers/web/user/notification.go
+++ b/routers/web/user/notification.go
@@ -180,12 +180,6 @@ func NotificationStatusPost(ctx *context.Context) {
 		ctx.Redirect(url, http.StatusSeeOther)
 	}
 
-	// convert the action to the displayed list
-	if statusStr == "unread" {
-		ctx.SetFormString("q", "read")
-	} else {
-		ctx.SetFormString("q", "unread")
-	}
 	getNotifications(ctx)
 	if ctx.Written() {
 		return
diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index 5dcd90159bd1c..98a7fe7e071a7 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -70,7 +70,7 @@
 							<div class="notifications-buttons tw-items-center tw-justify-end tw-gap-1 tw-px-1">
 								{{if ne .Status 3}}
 									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
-										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
 										hx-target="#notification_div"
 										hx-swap="outerHTML"
 										hx-vals='{"notification_id": "{{.ID}}", "status": "pinned", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true" }'
@@ -81,7 +81,7 @@
 								{{end}}
 								{{if or (eq .Status 1) (eq .Status 3)}}
 									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
-										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
 										hx-target="#notification_div"
 										hx-swap="outerHTML"
 										hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
@@ -90,7 +90,7 @@
 									</button>
 								{{else if eq .Status 2}}
 									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
-										hx-post="{{AppSubUrl}}/notifications/status"
+										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
 										hx-target="#notification_div"
 										hx-swap="outerHTML"
 										hx-vals='{"notification_id": "{{.ID}}", "status": "unread", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'

From 138cc99b4e2d79f4b88ce93db6078448bf3a5f25 Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 22:05:56 +0800
Subject: [PATCH 05/11] refactor

---
 routers/web/user/notification.go              |  73 +++-----
 .../user/notification/notification_div.tmpl   | 159 ++++++++----------
 web_src/css/user.css                          |  11 ++
 3 files changed, 106 insertions(+), 137 deletions(-)

diff --git a/routers/web/user/notification.go b/routers/web/user/notification.go
index 610a9b8076059..0bdd14f605079 100644
--- a/routers/web/user/notification.go
+++ b/routers/web/user/notification.go
@@ -4,7 +4,6 @@
 package user
 
 import (
-	"errors"
 	"fmt"
 	"net/http"
 	"net/url"
@@ -34,58 +33,42 @@ const (
 	tplNotificationSubscriptions templates.TplName = "user/notification/notification_subscriptions"
 )
 
-// Notifications is the notifications page
+// Notifications is the notification list page
 func Notifications(ctx *context.Context) {
-	getNotifications(ctx)
+	prepareUserNotificationsData(ctx)
 	if ctx.Written() {
 		return
 	}
 	if ctx.FormBool("div-only") {
-		ctx.Data["SequenceNumber"] = ctx.FormString("sequence-number")
 		ctx.HTML(http.StatusOK, tplNotificationDiv)
 		return
 	}
 	ctx.HTML(http.StatusOK, tplNotification)
 }
 
-func getNotifications(ctx *context.Context) {
-	var (
-		keyword = ctx.FormTrim("q")
-		status  activities_model.NotificationStatus
-		page    = ctx.FormInt("page")
-		perPage = ctx.FormInt("perPage")
-	)
-	if page < 1 {
-		page = 1
-	}
-	if perPage < 1 {
-		perPage = 20
-	}
-
-	switch keyword {
-	case "read":
-		status = activities_model.NotificationStatusRead
-	default:
-		status = activities_model.NotificationStatusUnread
-	}
+func prepareUserNotificationsData(ctx *context.Context) {
+	pageType := ctx.FormString("type", ctx.FormString("q")) // "q" is the legacy query parameter for "page type"
+	page := min(1, ctx.FormInt("page"))
+	perPage := util.IfZero(ctx.FormInt("perPage"), 20)
+	queryStatus := util.Iif(pageType == "read", activities_model.NotificationStatusRead, activities_model.NotificationStatusUnread)
 
 	total, err := db.Count[activities_model.Notification](ctx, activities_model.FindNotificationOptions{
 		UserID: ctx.Doer.ID,
-		Status: []activities_model.NotificationStatus{status},
+		Status: []activities_model.NotificationStatus{queryStatus},
 	})
 	if err != nil {
 		ctx.ServerError("ErrGetNotificationCount", err)
 		return
 	}
 
-	// redirect to last page if request page is more than total pages
+	// redirect to the last page if request page is more than total pages
 	pager := context.NewPagination(int(total), perPage, page, 5)
 	if pager.Paginater.Current() < page {
 		ctx.Redirect(fmt.Sprintf("%s/notifications?q=%s&page=%d", setting.AppSubURL, url.QueryEscape(ctx.FormString("q")), pager.Paginater.Current()))
 		return
 	}
 
-	statuses := []activities_model.NotificationStatus{status, activities_model.NotificationStatusPinned}
+	statuses := []activities_model.NotificationStatus{queryStatus, activities_model.NotificationStatusPinned}
 	nls, err := db.Find[activities_model.Notification](ctx, activities_model.FindNotificationOptions{
 		ListOptions: db.ListOptions{
 			PageSize: perPage,
@@ -142,51 +125,37 @@ func getNotifications(ctx *context.Context) {
 	}
 
 	ctx.Data["Title"] = ctx.Tr("notifications")
-	ctx.Data["Keyword"] = keyword
-	ctx.Data["Status"] = status
+	ctx.Data["PageType"] = pageType
 	ctx.Data["Notifications"] = notifications
-
+	ctx.Data["Link"] = setting.AppSubURL + "/notifications"
+	ctx.Data["SequenceNumber"] = ctx.FormString("sequence-number")
 	pager.AddParamFromRequest(ctx.Req)
 	ctx.Data["Page"] = pager
 }
 
 // NotificationStatusPost is a route for changing the status of a notification
 func NotificationStatusPost(ctx *context.Context) {
-	var (
-		notificationID = ctx.FormInt64("notification_id")
-		statusStr      = ctx.FormString("status")
-		status         activities_model.NotificationStatus
-	)
-
-	switch statusStr {
-	case "read":
+	notificationID := ctx.FormInt64("notification_id")
+	var status activities_model.NotificationStatus
+	switch ctx.FormString("notification_action") {
+	case "mark_as_read":
 		status = activities_model.NotificationStatusRead
-	case "unread":
+	case "mark_as_unread":
 		status = activities_model.NotificationStatusUnread
-	case "pinned":
+	case "pin":
 		status = activities_model.NotificationStatusPinned
 	default:
-		ctx.ServerError("InvalidNotificationStatus", errors.New("Invalid notification status"))
-		return
+		return // ignore user's invalid input
 	}
-
 	if _, err := activities_model.SetNotificationStatus(ctx, notificationID, ctx.Doer, status); err != nil {
 		ctx.ServerError("SetNotificationStatus", err)
 		return
 	}
 
-	if !ctx.FormBool("noredirect") {
-		url := fmt.Sprintf("%s/notifications?page=%s", setting.AppSubURL, url.QueryEscape(ctx.FormString("page")))
-		ctx.Redirect(url, http.StatusSeeOther)
-	}
-
-	getNotifications(ctx)
+	prepareUserNotificationsData(ctx)
 	if ctx.Written() {
 		return
 	}
-	ctx.Data["Link"] = setting.AppSubURL + "/notifications"
-	ctx.Data["SequenceNumber"] = ctx.Req.PostFormValue("sequence-number")
-
 	ctx.HTML(http.StatusOK, tplNotificationDiv)
 }
 
diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index 98a7fe7e071a7..e6c5090129ef6 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -1,108 +1,97 @@
 <div role="main" aria-label="{{.Title}}" class="page-content user notification" id="notification_div" data-sequence-number="{{.SequenceNumber}}">
 	<div class="ui container">
+		{{$statusUnread := 1}}{{$statusRead := 2}}{{$statusPinned := 3}}
 		{{$notificationUnreadCount := call .PageGlobalData.GetNotificationUnreadCount}}
-		<div class="tw-flex tw-items-center tw-justify-between tw-mb-[--page-spacing]">
+		{{$pageTypeIsRead := eq $.PageType "read"}}
+		<div class="flex-text-block tw-justify-between tw-mb-[--page-spacing]">
 			<div class="small-menu-items ui compact tiny menu">
-				<a class="{{if eq .Status 1}}active {{end}}item" href="{{AppSubUrl}}/notifications?q=unread">
+				<a class="{{if not $pageTypeIsRead}}active{{end}} item" href="{{AppSubUrl}}/notifications?type=unread">
 					{{ctx.Locale.Tr "notification.unread"}}
 					<div class="notifications-unread-count ui label {{if not $notificationUnreadCount}}tw-hidden{{end}}">{{$notificationUnreadCount}}</div>
 				</a>
-				<a class="{{if eq .Status 2}}active {{end}}item" href="{{AppSubUrl}}/notifications?q=read">
+				<a class="{{if $pageTypeIsRead}}active{{end}} item" href="{{AppSubUrl}}/notifications?type=read">
 					{{ctx.Locale.Tr "notification.read"}}
 				</a>
 			</div>
-			{{if and (eq .Status 1)}}
+			{{if and (not $pageTypeIsRead) $notificationUnreadCount}}
 				<form action="{{AppSubUrl}}/notifications/purge" method="post">
 					{{$.CsrfTokenHtml}}
-					<div class="{{if not $notificationUnreadCount}}tw-hidden{{end}}">
-						<button class="ui mini button primary tw-mr-0" title="{{ctx.Locale.Tr "notification.mark_all_as_read"}}">
-							{{svg "octicon-checklist"}}
-						</button>
-					</div>
+					<button class="ui mini button primary tw-mr-0" title="{{ctx.Locale.Tr "notification.mark_all_as_read"}}">
+						{{svg "octicon-checklist"}}
+					</button>
 				</form>
 			{{end}}
 		</div>
-		<div class="tw-p-0">
-			<div id="notification_table">
-				{{if not .Notifications}}
-					<div class="tw-flex tw-items-center tw-flex-col tw-p-4">
-						{{svg "octicon-inbox" 56 "tw-mb-4"}}
-						{{if eq .Status 1}}
-							{{ctx.Locale.Tr "notification.no_unread"}}
+		<div id="notification_table">
+			{{range $one := .Notifications}}
+				<div class="notifications-item" id="notification_{{$one.ID}}" data-status="{{$one.Status}}">
+					<div class="tw-self-start tw-mt-[2px]">
+						{{if $one.Issue}}
+							{{template "shared/issueicon" $one.Issue}}
 						{{else}}
-							{{ctx.Locale.Tr "notification.no_read"}}
+							{{svg "octicon-repo" 16 "text grey"}}
 						{{end}}
 					</div>
-				{{else}}
-					{{range $notification := .Notifications}}
-						<div class="notifications-item tw-flex tw-items-center tw-flex-wrap tw-gap-2 tw-p-2" id="notification_{{.ID}}" data-status="{{.Status}}">
-							<div class="notifications-icon tw-ml-2 tw-mr-1 tw-self-start tw-mt-1">
-								{{if .Issue}}
-									{{template "shared/issueicon" .Issue}}
-								{{else}}
-									{{svg "octicon-repo" 16 "text grey"}}
-								{{end}}
-							</div>
-							<a class="notifications-link tw-flex tw-flex-1 tw-flex-col silenced" href="{{.Link ctx}}">
-								<div class="notifications-top-row tw-text-13 tw-break-anywhere">
-									{{.Repository.FullName}} {{if .Issue}}<span class="text light-3">#{{.Issue.Index}}</span>{{end}}
-									{{if eq .Status 3}}
-										{{svg "octicon-pin" 13 "text blue tw-mt-0.5 tw-ml-1"}}
-									{{end}}
-								</div>
-								<div class="notifications-bottom-row tw-text-16 tw-py-0.5">
-									<span class="issue-title tw-break-anywhere">
-										{{if .Issue}}
-											{{.Issue.Title | ctx.RenderUtils.RenderIssueSimpleTitle}}
-										{{else}}
-											{{.Repository.FullName}}
-										{{end}}
-									</span>
-								</div>
-							</a>
-							<div class="notifications-updated tw-items-center tw-mr-2">
-								{{if .Issue}}
-									{{DateUtils.TimeSince .Issue.UpdatedUnix}}
-								{{else}}
-									{{DateUtils.TimeSince .UpdatedUnix}}
-								{{end}}
-							</div>
-							<div class="notifications-buttons tw-items-center tw-justify-end tw-gap-1 tw-px-1">
-								{{if ne .Status 3}}
-									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.pin"}}"
-										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
-										hx-target="#notification_div"
-										hx-swap="outerHTML"
-										hx-vals='{"notification_id": "{{.ID}}", "status": "pinned", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true" }'
-										{{if eq .Status 3}}disabled{{end}}
-										>
-										{{svg "octicon-pin"}}
-									</button>
-								{{end}}
-								{{if or (eq .Status 1) (eq .Status 3)}}
-									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_read"}}"
-										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
-										hx-target="#notification_div"
-										hx-swap="outerHTML"
-										hx-vals='{"notification_id": "{{.ID}}", "status": "read", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
-										>
-										{{svg "octicon-check"}}
-									</button>
-								{{else if eq .Status 2}}
-									<button type="button" class="btn interact-bg tw-p-2" title="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
-										hx-post="{{AppSubUrl}}/notifications/status?q={{ $.Keyword }}"
-										hx-target="#notification_div"
-										hx-swap="outerHTML"
-										hx-vals='{"notification_id": "{{.ID}}", "status": "unread", "page": "{{$.Page.Paginater.Current}}", "_csrf": "{{$.CsrfToken}}", "noredirect": "true"}'
-										>
-										{{svg "octicon-bell"}}
-									</button>
-								{{end}}
-							</div>
+					<a class="notifications-link silenced tw-flex-1" href="{{$one.Link ctx}}">
+						<div class="flex-text-block tw-text-[0.95em]">
+							{{$one.Repository.FullName}} {{if $one.Issue}}<span class="text light-3">#{{$one.Issue.Index}}</span>{{end}}
+							{{if eq $one.Status $statusPinned}}
+								{{svg "octicon-pin" 13 "text blue"}}
+							{{end}}
+						</div>
+						<div class="tw-text-16 tw-py-0.5">
+							{{if $one.Issue}}
+								{{$one.Issue.Title | ctx.RenderUtils.RenderIssueSimpleTitle}}
+							{{else}}
+								{{$one.Repository.FullName}}
+							{{end}}
 						</div>
+					</a>
+					<div class="notifications-updated flex-text-inline">
+						{{if $one.Issue}}
+							{{DateUtils.TimeSince $one.Issue.UpdatedUnix}}
+						{{else}}
+							{{DateUtils.TimeSince $one.UpdatedUnix}}
+						{{end}}
+					</div>
+					<form class="notifications-buttons" action="{{AppSubUrl}}/notifications/status?type={{$.PageType}}" method="post"
+								hx-boost="true" hx-target="#notification_div" hx-swap="outerHTML"
+					>
+						{{$.CsrfTokenHtml}}
+						<input type="hidden" name="notification_id" value="{{$one.ID}}">
+						<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
+						{{if ne $one.Status $statusPinned}}
+							<button class="btn interact-bg tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "notification.pin"}}"
+											name="notification_action" value="pin"
+							>
+								{{svg "octicon-pin"}}
+							</button>
+						{{end}}
+						{{if or (eq $one.Status $statusUnread) (eq $one.Status $statusPinned)}}
+							<button class="btn interact-bg tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "notification.mark_as_read"}}"
+											name="notification_action" value="mark_as_read"
+							>
+								{{svg "octicon-check"}}
+							</button>
+						{{else if eq $one.Status $statusRead}}
+							<button class="btn interact-bg tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "notification.mark_as_unread"}}"
+											name="notification_action" value="mark_as_unread"
+							>
+								{{svg "octicon-bell"}}
+							</button>
+						{{end}}
+					</form>
+				</div>
+			{{else}}
+				<div class="empty-placeholder">
+					{{svg "octicon-inbox" 56 "tw-mb-4"}}
+					{{if eq .Status 1}}
+						{{ctx.Locale.Tr "notification.no_unread"}}
+					{{else}}
+						{{ctx.Locale.Tr "notification.no_read"}}
 					{{end}}
-				{{end}}
-			</div>
+				</div>
+			{{end}}
 		</div>
 		{{template "base/paginate" .}}
 	</div>
diff --git a/web_src/css/user.css b/web_src/css/user.css
index caabf1834cbb2..d42e8688fbc68 100644
--- a/web_src/css/user.css
+++ b/web_src/css/user.css
@@ -114,6 +114,14 @@
   border-radius: var(--border-radius);
 }
 
+.notifications-item {
+  display: flex;
+  align-items: center;
+  flex-wrap: wrap;
+  gap: 0.5em;
+  padding: 0.5em 1em;
+}
+
 .notifications-item:hover {
   background: var(--color-hover);
 }
@@ -129,6 +137,9 @@
 
 .notifications-item:hover .notifications-buttons {
   display: flex;
+  align-items: center;
+  justify-content: end;
+  gap: 0.25em;
 }
 
 .notifications-item:hover .notifications-updated {

From cb0c1068af0ccfd10c70d1a78270a4d4f7d73861 Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 22:20:34 +0800
Subject: [PATCH 06/11] fix paging

---
 routers/web/user/notification.go                 | 16 ++++++++--------
 .../user/notification/notification_div.tmpl      |  3 +--
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/routers/web/user/notification.go b/routers/web/user/notification.go
index 0bdd14f605079..bebd120fd1f6c 100644
--- a/routers/web/user/notification.go
+++ b/routers/web/user/notification.go
@@ -48,8 +48,8 @@ func Notifications(ctx *context.Context) {
 
 func prepareUserNotificationsData(ctx *context.Context) {
 	pageType := ctx.FormString("type", ctx.FormString("q")) // "q" is the legacy query parameter for "page type"
-	page := min(1, ctx.FormInt("page"))
-	perPage := util.IfZero(ctx.FormInt("perPage"), 20)
+	page := max(1, ctx.FormInt("page"))
+	perPage := util.IfZero(ctx.FormInt("perPage"), 20) // this value is never used or exposed ....
 	queryStatus := util.Iif(pageType == "read", activities_model.NotificationStatusRead, activities_model.NotificationStatusUnread)
 
 	total, err := db.Count[activities_model.Notification](ctx, activities_model.FindNotificationOptions{
@@ -64,7 +64,7 @@ func prepareUserNotificationsData(ctx *context.Context) {
 	// redirect to the last page if request page is more than total pages
 	pager := context.NewPagination(int(total), perPage, page, 5)
 	if pager.Paginater.Current() < page {
-		ctx.Redirect(fmt.Sprintf("%s/notifications?q=%s&page=%d", setting.AppSubURL, url.QueryEscape(ctx.FormString("q")), pager.Paginater.Current()))
+		ctx.Redirect(fmt.Sprintf("%s/notifications?type=%s&page=%d", setting.AppSubURL, url.QueryEscape(pageType), pager.Paginater.Current()))
 		return
 	}
 
@@ -136,18 +136,18 @@ func prepareUserNotificationsData(ctx *context.Context) {
 // NotificationStatusPost is a route for changing the status of a notification
 func NotificationStatusPost(ctx *context.Context) {
 	notificationID := ctx.FormInt64("notification_id")
-	var status activities_model.NotificationStatus
+	var newStatus activities_model.NotificationStatus
 	switch ctx.FormString("notification_action") {
 	case "mark_as_read":
-		status = activities_model.NotificationStatusRead
+		newStatus = activities_model.NotificationStatusRead
 	case "mark_as_unread":
-		status = activities_model.NotificationStatusUnread
+		newStatus = activities_model.NotificationStatusUnread
 	case "pin":
-		status = activities_model.NotificationStatusPinned
+		newStatus = activities_model.NotificationStatusPinned
 	default:
 		return // ignore user's invalid input
 	}
-	if _, err := activities_model.SetNotificationStatus(ctx, notificationID, ctx.Doer, status); err != nil {
+	if _, err := activities_model.SetNotificationStatus(ctx, notificationID, ctx.Doer, newStatus); err != nil {
 		ctx.ServerError("SetNotificationStatus", err)
 		return
 	}
diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index e6c5090129ef6..a746d7576f082 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -54,12 +54,11 @@
 							{{DateUtils.TimeSince $one.UpdatedUnix}}
 						{{end}}
 					</div>
-					<form class="notifications-buttons" action="{{AppSubUrl}}/notifications/status?type={{$.PageType}}" method="post"
+					<form class="notifications-buttons" action="{{AppSubUrl}}/notifications/status?type={{$.PageType}}&page={{$.Page.Paginater.Current}}&perPage={{$.Page.Paginater.PagingNum}}" method="post"
 								hx-boost="true" hx-target="#notification_div" hx-swap="outerHTML"
 					>
 						{{$.CsrfTokenHtml}}
 						<input type="hidden" name="notification_id" value="{{$one.ID}}">
-						<input type="hidden" name="page" value="{{$.Page.Paginater.Current}}">
 						{{if ne $one.Status $statusPinned}}
 							<button class="btn interact-bg tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "notification.pin"}}"
 											name="notification_action" value="pin"

From c3247a569a66f2d10041736cf6067d0ba543066c Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 22:26:02 +0800
Subject: [PATCH 07/11] fix redirect

---
 routers/web/user/notification.go | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/routers/web/user/notification.go b/routers/web/user/notification.go
index bebd120fd1f6c..aaf9d435c09cd 100644
--- a/routers/web/user/notification.go
+++ b/routers/web/user/notification.go
@@ -6,7 +6,6 @@ package user
 import (
 	"fmt"
 	"net/http"
-	"net/url"
 	"strings"
 
 	activities_model "code.gitea.io/gitea/models/activities"
@@ -61,11 +60,11 @@ func prepareUserNotificationsData(ctx *context.Context) {
 		return
 	}
 
-	// redirect to the last page if request page is more than total pages
 	pager := context.NewPagination(int(total), perPage, page, 5)
 	if pager.Paginater.Current() < page {
-		ctx.Redirect(fmt.Sprintf("%s/notifications?type=%s&page=%d", setting.AppSubURL, url.QueryEscape(pageType), pager.Paginater.Current()))
-		return
+		// use the last page if the requested page is more than total pages
+		page = pager.Paginater.Current()
+		pager = context.NewPagination(int(total), perPage, page, 5)
 	}
 
 	statuses := []activities_model.NotificationStatus{queryStatus, activities_model.NotificationStatusPinned}

From 45f1d57d7b923fe7d381ca365087a5b68e49763d Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 22:49:29 +0800
Subject: [PATCH 08/11] fix SSE ...

---
 web_src/js/features/notification.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/web_src/js/features/notification.ts b/web_src/js/features/notification.ts
index dc0acb0244edd..206d31ad4b0d2 100644
--- a/web_src/js/features/notification.ts
+++ b/web_src/js/features/notification.ts
@@ -33,8 +33,7 @@ export function initNotificationsTable() {
 
 async function receiveUpdateCount(event: MessageEvent) {
   try {
-    const data = JSON.parse(event.data);
-
+    const data = event.data;
     for (const count of document.querySelectorAll('.notification_count')) {
       count.classList.toggle('tw-hidden', data.Count === 0);
       count.textContent = `${data.Count}`;

From a646b328b8ca79fc052c162fdcf0929a4d49e060 Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 23:01:22 +0800
Subject: [PATCH 09/11] fix htmx process, remove bfcache support

---
 web_src/js/features/notification.ts | 35 +++++------------------------
 web_src/js/index-domready.ts        |  3 +--
 2 files changed, 6 insertions(+), 32 deletions(-)

diff --git a/web_src/js/features/notification.ts b/web_src/js/features/notification.ts
index 206d31ad4b0d2..6812119328270 100644
--- a/web_src/js/features/notification.ts
+++ b/web_src/js/features/notification.ts
@@ -1,36 +1,10 @@
 import {GET} from '../modules/fetch.ts';
-import {toggleElem, type DOMEvent, createElementFromHTML} from '../utils/dom.ts';
+import {toggleElem, createElementFromHTML} from '../utils/dom.ts';
 import {logoutFromWorker} from '../modules/worker.ts';
 
 const {appSubUrl, notificationSettings, assetVersionEncoded} = window.config;
 let notificationSequenceNumber = 0;
 
-export function initNotificationsTable() {
-  const table = document.querySelector('#notification_table');
-  if (!table) return;
-
-  // when page restores from bfcache, delete previously clicked items
-  window.addEventListener('pageshow', (e) => {
-    if (e.persisted) { // page was restored from bfcache
-      const table = document.querySelector('#notification_table');
-      const unreadCountEl = document.querySelector<HTMLElement>('.notifications-unread-count');
-      let unreadCount = parseInt(unreadCountEl.textContent);
-      for (const item of table.querySelectorAll('.notifications-item[data-remove="true"]')) {
-        item.remove();
-        unreadCount -= 1;
-      }
-      unreadCountEl.textContent = String(unreadCount);
-    }
-  });
-
-  // mark clicked unread links for deletion on bfcache restore
-  for (const link of table.querySelectorAll<HTMLAnchorElement>('.notifications-item[data-status="1"] .notifications-link')) {
-    link.addEventListener('click', (e: DOMEvent<MouseEvent>) => {
-      e.target.closest('.notifications-item').setAttribute('data-remove', 'true');
-    });
-  }
-}
-
 async function receiveUpdateCount(event: MessageEvent) {
   try {
     const data = event.data;
@@ -143,11 +117,11 @@ async function updateNotificationCountWithCallback(callback: (timeout: number, n
 }
 
 async function updateNotificationTable() {
-  const notificationDiv = document.querySelector('#notification_div');
+  let notificationDiv = document.querySelector('#notification_div');
   if (notificationDiv) {
     try {
       const params = new URLSearchParams(window.location.search);
-      params.set('div-only', String(true));
+      params.set('div-only', 'true');
       params.set('sequence-number', String(++notificationSequenceNumber));
       const response = await GET(`${appSubUrl}/notifications?${params.toString()}`);
 
@@ -159,7 +133,8 @@ async function updateNotificationTable() {
       const el = createElementFromHTML(data);
       if (parseInt(el.getAttribute('data-sequence-number')) === notificationSequenceNumber) {
         notificationDiv.outerHTML = data;
-        initNotificationsTable();
+        notificationDiv = document.querySelector('#notification_div');
+        window.htmx.process(notificationDiv); // when using htmx, we must always remember to process the new content changed by us
       }
     } catch (error) {
       console.error(error);
diff --git a/web_src/js/index-domready.ts b/web_src/js/index-domready.ts
index ca18d1e8281f5..770c7fc00c642 100644
--- a/web_src/js/index-domready.ts
+++ b/web_src/js/index-domready.ts
@@ -15,7 +15,7 @@ import {initTableSort} from './features/tablesort.ts';
 import {initAdminUserListSearchForm} from './features/admin/users.ts';
 import {initAdminConfigs} from './features/admin/config.ts';
 import {initMarkupAnchors} from './markup/anchors.ts';
-import {initNotificationCount, initNotificationsTable} from './features/notification.ts';
+import {initNotificationCount} from './features/notification.ts';
 import {initRepoIssueContentHistory} from './features/repo-issue-content.ts';
 import {initStopwatch} from './features/stopwatch.ts';
 import {initFindFileInRepo} from './features/repo-findfile.ts';
@@ -117,7 +117,6 @@ const initPerformanceTracer = callInitFunctions([
   initDashboardRepoList,
 
   initNotificationCount,
-  initNotificationsTable,
 
   initOrgTeam,
 

From 11443d8e0fb79a89b58a84bf55de5402540326ff Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 23:15:05 +0800
Subject: [PATCH 10/11] fix empty notice

---
 templates/user/notification/notification_div.tmpl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/templates/user/notification/notification_div.tmpl b/templates/user/notification/notification_div.tmpl
index a746d7576f082..b9faba2b90b20 100644
--- a/templates/user/notification/notification_div.tmpl
+++ b/templates/user/notification/notification_div.tmpl
@@ -84,10 +84,10 @@
 			{{else}}
 				<div class="empty-placeholder">
 					{{svg "octicon-inbox" 56 "tw-mb-4"}}
-					{{if eq .Status 1}}
-						{{ctx.Locale.Tr "notification.no_unread"}}
-					{{else}}
+					{{if $pageTypeIsRead}}
 						{{ctx.Locale.Tr "notification.no_read"}}
+					{{else}}
+						{{ctx.Locale.Tr "notification.no_unread"}}
 					{{end}}
 				</div>
 			{{end}}

From b0c11107ed7689068cfac33c4c5c60596649575e Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Wed, 9 Jul 2025 23:23:46 +0800
Subject: [PATCH 11/11] use ts strict type

---
 web_src/js/features/notification.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/web_src/js/features/notification.ts b/web_src/js/features/notification.ts
index 6812119328270..4a1aa3ede94af 100644
--- a/web_src/js/features/notification.ts
+++ b/web_src/js/features/notification.ts
@@ -5,9 +5,9 @@ import {logoutFromWorker} from '../modules/worker.ts';
 const {appSubUrl, notificationSettings, assetVersionEncoded} = window.config;
 let notificationSequenceNumber = 0;
 
-async function receiveUpdateCount(event: MessageEvent) {
+async function receiveUpdateCount(event: MessageEvent<{type: string, data: string}>) {
   try {
-    const data = event.data;
+    const data = JSON.parse(event.data.data);
     for (const count of document.querySelectorAll('.notification_count')) {
       count.classList.toggle('tw-hidden', data.Count === 0);
       count.textContent = `${data.Count}`;
@@ -44,7 +44,7 @@ export function initNotificationCount() {
       type: 'start',
       url: `${window.location.origin}${appSubUrl}/user/events`,
     });
-    worker.port.addEventListener('message', (event: MessageEvent) => {
+    worker.port.addEventListener('message', (event: MessageEvent<{type: string, data: string}>) => {
       if (!event.data || !event.data.type) {
         console.error('unknown worker message event', event);
         return;