fix

go-gitea · Jun 18, 2024 · 3c8533f · 3c8533f
1 parent 17baf1a
commit 3c8533f
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 5 deletions.
diff --git a/public/assets/img/404.png b/public/assets/img/404.png
diff --git a/public/assets/img/500.png b/public/assets/img/500.png
diff --git a/templates/status/500.tmpl b/templates/status/500.tmpl
@@ -34,14 +34,14 @@
 				<style> .ui.message.flash-message { text-align: left; } </style>
 				{{template "base/alert" .}}
 			</div>
-			<p class="tw-mt-8 center"><img src="{{AssetUrlPrefix}}/img/500.png" alt="Internal Server Error"></p>
+			<div class="status-page-error-code">500 Internal Server Error</div>
 			<div class="divider"></div>
 			<div class="ui container tw-my-8">
 				{{if .ErrorMsg}}
 					<p>{{ctx.Locale.Tr "error.occurred"}}:</p>
 					<pre class="tw-whitespace-pre-wrap tw-break-all">{{.ErrorMsg}}</pre>
 				{{end}}
-				<div class="center tw-mt-8">
+				<div class="tw-mt-8 tw-text-center">
 					{{if or .SignedUser.IsAdmin .ShowFooterVersion}}<p>{{ctx.Locale.Tr "admin.config.app_ver"}}: {{AppVer}}</p>{{end}}
 					{{if .SignedUser.IsAdmin}}<p>{{ctx.Locale.Tr "error.report_message"}}</p>{{end}}
 				</div>

diff --git a/tests/integration/compare_test.go b/tests/integration/compare_test.go
@@ -4,6 +4,7 @@
 package integration
 
 import (
+	"code.gitea.io/gitea/modules/test"
 	"fmt"
 	"net/http"
 	"net/url"
@@ -32,7 +33,7 @@ func TestCompareTag(t *testing.T) {
 
 	req = NewRequest(t, "GET", "/user2/repo1/compare/invalid")
 	resp = session.MakeRequest(t, req, http.StatusNotFound)
-	assert.False(t, strings.Contains(resp.Body.String(), "/assets/img/500.png"), "expect 404 page not 500")
+	assert.True(t, test.IsNormalPageCompleted(resp.Body.String()), "expect 404 page not 500")
 }
 
 // Compare with inferred default branch (master)

diff --git a/tests/integration/links_test.go b/tests/integration/links_test.go
@@ -37,8 +37,6 @@ func TestLinksNoLogin(t *testing.T) {
 		"/user2/repo1/projects",
 		"/user2/repo1/projects/1",
 		"/user2/repo1/releases/tag/delete-tag", // It's the only one existing record on release.yml which has is_tag: true
-		"/assets/img/404.png",
-		"/assets/img/500.png",
 		"/.well-known/security.txt",
 	}
 

diff --git a/web_src/css/base.css b/web_src/css/base.css
@@ -477,6 +477,15 @@ img.ui.avatar,
   padding-bottom: 80px;
 }
 
+.status-page-error-code {
+  font-size: 3em;
+  line-height: initial;
+  text-align: center;
+  padding: 2em 0;
+  font-weight: var(--font-weight-bold);
+  color: var(--color-text-light-3);
+}
+
 /* add margin below .secondary nav when it is the first child */
 .page-content > :first-child.secondary-nav {
   margin-bottom: 14px;