From 0a201f4b5e42f32a53f0da38d0ebc6dd93b77fc5 Mon Sep 17 00:00:00 2001
From: 6543 <6543@obermui.de>
Date: Sun, 21 Jan 2024 14:10:33 +0100
Subject: [PATCH 1/2] Revert "Don't do a full page load when clicking the
 follow button (#28872)"

This reverts commit 1df06e3f399307c14b60a2b88e1b26cedc1ae2f9.
---
 routers/web/user/profile.go                   | 11 ++---------
 templates/shared/user/profile_big_avatar.tmpl |  8 ++++----
 2 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/routers/web/user/profile.go b/routers/web/user/profile.go
index 73ab93caed5a..c5305ebcd9f4 100644
--- a/routers/web/user/profile.go
+++ b/routers/web/user/profile.go
@@ -14,7 +14,6 @@ import (
 	"code.gitea.io/gitea/models/db"
 	repo_model "code.gitea.io/gitea/models/repo"
 	user_model "code.gitea.io/gitea/models/user"
-	"code.gitea.io/gitea/modules/base"
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/git"
 	"code.gitea.io/gitea/modules/log"
@@ -27,10 +26,6 @@ import (
 	shared_user "code.gitea.io/gitea/routers/web/shared/user"
 )
 
-const (
-	tplProfileBigAvatar base.TplName = "shared/user/profile_big_avatar"
-)
-
 // OwnerProfile render profile page for a user or a organization (aka, repo owner)
 func OwnerProfile(ctx *context.Context) {
 	if strings.Contains(ctx.Req.Header.Get("Accept"), "application/rss+xml") {
@@ -314,10 +309,8 @@ func Action(ctx *context.Context) {
 
 	if err != nil {
 		log.Error("Failed to apply action %q: %v", ctx.FormString("action"), err)
-		ctx.Error(http.StatusBadRequest, fmt.Sprintf("Action %q failed", ctx.FormString("action")))
+		ctx.JSONError(fmt.Sprintf("Action %q failed", ctx.FormString("action")))
 		return
 	}
-
-	shared_user.PrepareContextForProfileBigAvatar(ctx)
-	ctx.HTML(http.StatusOK, tplProfileBigAvatar)
+	ctx.JSONOK()
 }
diff --git a/templates/shared/user/profile_big_avatar.tmpl b/templates/shared/user/profile_big_avatar.tmpl
index 7afc852d1b57..a637a9a5f915 100644
--- a/templates/shared/user/profile_big_avatar.tmpl
+++ b/templates/shared/user/profile_big_avatar.tmpl
@@ -1,4 +1,4 @@
-<div id="profile-avatar-card" class="ui card">
+<div class="ui card">
 	<div id="profile-avatar" class="content gt-df">
 	{{if eq .SignedUserID .ContextUser.ID}}
 		<a class="image" href="{{AppSubUrl}}/user/settings" data-tooltip-content="{{ctx.Locale.Tr "user.change_avatar"}}">
@@ -110,13 +110,13 @@
 			</li>
 			{{end}}
 			{{if and .IsSigned (ne .SignedUserID .ContextUser.ID)}}
-			<li class="follow" hx-headers='{"x-csrf-token": "{{.CsrfToken}}"}' hx-target="#profile-avatar-card" hx-swap="outerHTML">
+			<li class="follow">
 				{{if $.IsFollowing}}
-					<button hx-post="{{.ContextUser.HomeLink}}?action=unfollow" class="ui basic red button">
+					<button class="ui basic red button link-action" data-url="{{.ContextUser.HomeLink}}?action=unfollow">
 						{{svg "octicon-person"}} {{ctx.Locale.Tr "user.unfollow"}}
 					</button>
 				{{else}}
-					<button hx-post="{{.ContextUser.HomeLink}}?action=follow" class="ui basic primary button">
+					<button class="ui basic primary button link-action" data-url="{{.ContextUser.HomeLink}}?action=follow">
 						{{svg "octicon-person"}} {{ctx.Locale.Tr "user.follow"}}
 					</button>
 				{{end}}

From 9f2f7c9cb224a1756663415c6b1d399af002aa5d Mon Sep 17 00:00:00 2001
From: 6543 <6543@obermui.de>
Date: Sun, 21 Jan 2024 14:10:53 +0100
Subject: [PATCH 2/2] Revert "Don't do a full page load when clicking the
 subscribe button (#28871)"

This reverts commit 14f6fcf4481cedc2cd9d5ea5aaff88e109089e1f.
---
 package-lock.json                               |  6 ------
 package.json                                    |  1 -
 routers/web/repo/issue_watch.go                 |  9 +--------
 templates/repo/issue/view_content/sidebar.tmpl  | 14 +++++++++++++-
 templates/repo/issue/view_content/watching.tmpl | 13 -------------
 web_src/js/features/common-global.js            |  1 -
 webpack.config.js                               |  1 -
 7 files changed, 14 insertions(+), 31 deletions(-)
 delete mode 100644 templates/repo/issue/view_content/watching.tmpl

diff --git a/package-lock.json b/package-lock.json
index d73fc5df5a8f..b299967eba59 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -26,7 +26,6 @@
         "esbuild-loader": "4.0.2",
         "escape-goat": "4.0.0",
         "fast-glob": "3.3.2",
-        "htmx.org": "1.9.10",
         "jquery": "3.7.1",
         "katex": "0.16.9",
         "license-checker-webpack-plugin": "0.2.1",
@@ -6159,11 +6158,6 @@
         "url": "https://github.com/fb55/entities?sponsor=1"
       }
     },
-    "node_modules/htmx.org": {
-      "version": "1.9.10",
-      "resolved": "https://registry.npmjs.org/htmx.org/-/htmx.org-1.9.10.tgz",
-      "integrity": "sha512-UgchasltTCrTuU2DQLom3ohHrBvwr7OqpwyAVJ9VxtNBng4XKkVsqrv0Qr3srqvM9ZNI3f1MmvVQQqK7KW/bTA=="
-    },
     "node_modules/http-proxy-agent": {
       "version": "7.0.0",
       "resolved": "https://registry.npmjs.org/http-proxy-agent/-/http-proxy-agent-7.0.0.tgz",
diff --git a/package.json b/package.json
index 5c6de1dd628b..801e85db83a7 100644
--- a/package.json
+++ b/package.json
@@ -25,7 +25,6 @@
     "esbuild-loader": "4.0.2",
     "escape-goat": "4.0.0",
     "fast-glob": "3.3.2",
-    "htmx.org": "1.9.10",
     "jquery": "3.7.1",
     "katex": "0.16.9",
     "license-checker-webpack-plugin": "0.2.1",
diff --git a/routers/web/repo/issue_watch.go b/routers/web/repo/issue_watch.go
index 1f51ceba5e47..1cb5cc7162d8 100644
--- a/routers/web/repo/issue_watch.go
+++ b/routers/web/repo/issue_watch.go
@@ -8,15 +8,10 @@ import (
 	"strconv"
 
 	issues_model "code.gitea.io/gitea/models/issues"
-	"code.gitea.io/gitea/modules/base"
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/log"
 )
 
-const (
-	tplWatching base.TplName = "repo/issue/view_content/watching"
-)
-
 // IssueWatch sets issue watching
 func IssueWatch(ctx *context.Context) {
 	issue := GetActionIssue(ctx)
@@ -57,7 +52,5 @@ func IssueWatch(ctx *context.Context) {
 		return
 	}
 
-	ctx.Data["Issue"] = issue
-	ctx.Data["IssueWatch"] = &issues_model.IssueWatch{IsWatching: watch}
-	ctx.HTML(http.StatusOK, tplWatching)
+	ctx.Redirect(issue.Link())
 }
diff --git a/templates/repo/issue/view_content/sidebar.tmpl b/templates/repo/issue/view_content/sidebar.tmpl
index 6c13eef02375..4334e4bcbdc2 100644
--- a/templates/repo/issue/view_content/sidebar.tmpl
+++ b/templates/repo/issue/view_content/sidebar.tmpl
@@ -270,7 +270,19 @@
 		<div class="ui watching">
 			<span class="text"><strong>{{ctx.Locale.Tr "notification.notifications"}}</strong></span>
 			<div class="gt-mt-3">
-				{{template "repo/issue/view_content/watching" .}}
+				<form method="post" action="{{.Issue.Link}}/watch">
+					<input type="hidden" name="watch" value="{{if $.IssueWatch.IsWatching}}0{{else}}1{{end}}">
+					{{$.CsrfTokenHtml}}
+					<button class="fluid ui button">
+						{{if $.IssueWatch.IsWatching}}
+							{{svg "octicon-mute" 16 "gt-mr-3"}}
+							{{ctx.Locale.Tr "repo.issues.unsubscribe"}}
+						{{else}}
+							{{svg "octicon-unmute" 16 "gt-mr-3"}}
+							{{ctx.Locale.Tr "repo.issues.subscribe"}}
+						{{end}}
+					</button>
+				</form>
 			</div>
 		</div>
 	{{end}}
diff --git a/templates/repo/issue/view_content/watching.tmpl b/templates/repo/issue/view_content/watching.tmpl
deleted file mode 100644
index 06b9d9af3325..000000000000
--- a/templates/repo/issue/view_content/watching.tmpl
+++ /dev/null
@@ -1,13 +0,0 @@
-<form hx-boost="true" hx-sync="this:replace" hx-target="this" hx-push-url="false" hx-swap="show:no-scroll" method="post" action="{{.Issue.Link}}/watch">
-	<input type="hidden" name="watch" value="{{if $.IssueWatch.IsWatching}}0{{else}}1{{end}}">
-	{{$.CsrfTokenHtml}}
-	<button class="fluid ui button">
-		{{if $.IssueWatch.IsWatching}}
-			{{svg "octicon-mute" 16 "gt-mr-3"}}
-			{{ctx.Locale.Tr "repo.issues.unsubscribe"}}
-		{{else}}
-			{{svg "octicon-unmute" 16 "gt-mr-3"}}
-			{{ctx.Locale.Tr "repo.issues.subscribe"}}
-		{{end}}
-	</button>
-</form>
diff --git a/web_src/js/features/common-global.js b/web_src/js/features/common-global.js
index ffa0434cffa0..0b00eb8e8ea7 100644
--- a/web_src/js/features/common-global.js
+++ b/web_src/js/features/common-global.js
@@ -12,7 +12,6 @@ import {showTemporaryTooltip} from '../modules/tippy.js';
 import {confirmModal} from './comp/ConfirmModal.js';
 import {showErrorToast} from '../modules/toast.js';
 import {request, POST} from '../modules/fetch.js';
-import 'htmx.org';
 
 const {appUrl, appSubUrl, csrfToken, i18n} = window.config;
 
diff --git a/webpack.config.js b/webpack.config.js
index 0d8418938f0c..448dc640036c 100644
--- a/webpack.config.js
+++ b/webpack.config.js
@@ -214,7 +214,6 @@ export default {
       },
       override: {
         'khroma@*': {licenseName: 'MIT'}, // https://github.com/fabiospampinato/khroma/pull/33
-        'htmx.org@1.9.10': {licenseName: 'BSD-2-Clause'}, // "BSD 2-Clause" -> "BSD-2-Clause"
       },
       emitError: true,
       allow: '(Apache-2.0 OR BSD-2-Clause OR BSD-3-Clause OR MIT OR ISC OR CPAL-1.0 OR Unlicense OR EPL-1.0 OR EPL-2.0)',