From 26d5028daca2fa4bfee24841e41cded668f5544a Mon Sep 17 00:00:00 2001
From: Mike L <cl.jeremy@qq.com>
Date: Sun, 14 Mar 2021 12:05:05 +0100
Subject: [PATCH 1/2] Re-enable listing of forks when logged out

---
 templates/repo/header.tmpl    | 4 ++--
 web_src/less/_base.less       | 1 +
 web_src/less/_repository.less | 4 ++++
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/templates/repo/header.tmpl b/templates/repo/header.tmpl
index f9df4062747b3..dbe447e3d84a9 100644
--- a/templates/repo/header.tmpl
+++ b/templates/repo/header.tmpl
@@ -83,8 +83,8 @@
 						</div>
 					</form>
 					{{if and (not .IsEmpty) ($.Permission.CanRead $.UnitTypeCode)}}
-						<div class="ui labeled button {{if not $.IsSigned}} disabled{{end}}" tabindex="0">
-							<a class="ui compact small basic button {{if or (not $.IsSigned) (not $.CanSignedUserFork)}}poping up{{end}}" {{if $.CanSignedUserFork}}href="{{AppSubUrl}}/repo/fork/{{.ID}}"{{else if $.IsSigned}} data-content="{{$.i18n.Tr "repo.fork_from_self"}}" {{ else }} data-content="{{$.i18n.Tr "repo.fork_guest_user" }}" rel="nofollow" href="{{AppSubUrl}}/user/login?redirect_to={{AppSubUrl}}/repo/fork/{{.ID}}" {{end}} data-position="top center" data-variation="tiny">
+						<div class="ui labeled button{{if not $.IsSigned}} disabled{{end}}" tabindex="0">
+							<a class="ui compact small basic button{{if not $.CanSignedUserFork}} poping up{{end}}"{{if $.CanSignedUserFork}} href="{{AppSubUrl}}/repo/fork/{{.ID}}"{{else if $.IsSigned}} data-content="{{$.i18n.Tr "repo.fork_from_self"}}" {{else}} data-content="{{$.i18n.Tr "repo.fork_guest_user"}}"{{end}} data-position="top center" data-variation="tiny">
 								{{svg "octicon-repo-forked"}}{{$.i18n.Tr "repo.fork"}}
 							</a>
 							<a class="ui basic label" href="{{.Link}}/forks">
diff --git a/web_src/less/_base.less b/web_src/less/_base.less
index 8c99fa4349e56..157f2610a6fd2 100644
--- a/web_src/less/_base.less
+++ b/web_src/less/_base.less
@@ -1585,6 +1585,7 @@ a.ui.label:hover {
   border-left: none;
 }
 
+.ui.labeled.button.disabled > .button:hover,
 .ui.basic.buttons .button,
 .ui.basic.button {
   color: var(--color-text-light);
diff --git a/web_src/less/_repository.less b/web_src/less/_repository.less
index 0984dd7cf3a6f..0f29b366704cd 100644
--- a/web_src/less/_repository.less
+++ b/web_src/less/_repository.less
@@ -2861,6 +2861,10 @@ tbody.commit-list {
   opacity: var(--opacity-disabled);
 }
 
+.repo-buttons .ui.labeled.button.disabled {
+  pointer-events: inherit !important;
+}
+
 .repo-buttons .ui.labeled.button > .label {
   border-left: 0 !important;
   margin: 0 !important;

From 9a623e5e6e58bc794abea82a42db5ea2c15c3080 Mon Sep 17 00:00:00 2001
From: Mike L <cl.jeremy@qq.com>
Date: Sun, 14 Mar 2021 14:27:35 +0100
Subject: [PATCH 2/2] Further improvements on repo button logic

---
 templates/repo/header.tmpl    |  4 ++--
 web_src/less/_base.less       |  2 +-
 web_src/less/_repository.less | 21 ++++++++++++++-------
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/templates/repo/header.tmpl b/templates/repo/header.tmpl
index dbe447e3d84a9..71963d698c71f 100644
--- a/templates/repo/header.tmpl
+++ b/templates/repo/header.tmpl
@@ -83,8 +83,8 @@
 						</div>
 					</form>
 					{{if and (not .IsEmpty) ($.Permission.CanRead $.UnitTypeCode)}}
-						<div class="ui labeled button{{if not $.IsSigned}} disabled{{end}}" tabindex="0">
-							<a class="ui compact small basic button{{if not $.CanSignedUserFork}} poping up{{end}}"{{if $.CanSignedUserFork}} href="{{AppSubUrl}}/repo/fork/{{.ID}}"{{else if $.IsSigned}} data-content="{{$.i18n.Tr "repo.fork_from_self"}}" {{else}} data-content="{{$.i18n.Tr "repo.fork_guest_user"}}"{{end}} data-position="top center" data-variation="tiny">
+						<div class="ui labeled button{{if not $.CanSignedUserFork}} poping up disabled{{end}}"{{if and (not $.CanSignedUserFork) $.IsSigned}} data-content="{{$.i18n.Tr "repo.fork_from_self"}}" {{else if not $.IsSigned}} data-content="{{$.i18n.Tr "repo.fork_guest_user"}}"{{end}} data-position="top center" data-variation="tiny" tabindex="0">
+							<a class="ui compact small basic button"{{if $.CanSignedUserFork}} href="{{AppSubUrl}}/repo/fork/{{.ID}}"{{end}}>
 								{{svg "octicon-repo-forked"}}{{$.i18n.Tr "repo.fork"}}
 							</a>
 							<a class="ui basic label" href="{{.Link}}/forks">
diff --git a/web_src/less/_base.less b/web_src/less/_base.less
index 157f2610a6fd2..351dbf5da013e 100644
--- a/web_src/less/_base.less
+++ b/web_src/less/_base.less
@@ -1585,7 +1585,7 @@ a.ui.label:hover {
   border-left: none;
 }
 
-.ui.labeled.button.disabled > .button:hover,
+.ui.labeled.button.disabled > .button,
 .ui.basic.buttons .button,
 .ui.basic.button {
   color: var(--color-text-light);
diff --git a/web_src/less/_repository.less b/web_src/less/_repository.less
index 0f29b366704cd..5a8476322c900 100644
--- a/web_src/less/_repository.less
+++ b/web_src/less/_repository.less
@@ -2856,18 +2856,25 @@ tbody.commit-list {
   align-items: center;
 }
 
-.repo-buttons button[disabled],
 .repo-buttons button[disabled] ~ .label {
   opacity: var(--opacity-disabled);
 }
 
-.repo-buttons .ui.labeled.button.disabled {
-  pointer-events: inherit !important;
-}
+.repo-buttons .ui.labeled.button {
+  cursor: initial;
 
-.repo-buttons .ui.labeled.button > .label {
-  border-left: 0 !important;
-  margin: 0 !important;
+  > .label {
+    border-left: 0 !important;
+    margin: 0 !important;
+  }
+
+  &.disabled {
+    pointer-events: inherit !important;
+
+    > .button {
+      pointer-events: none !important;
+    }
+  }
 }
 
 .tag-code {