From bfa160fc98a23923b6ce1cd4d99e8970d937d6ec Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Fri, 22 Mar 2024 01:04:03 +0800 Subject: [PATCH] Refactor repo header/list (#29969) 1. Use general "mobile-only" and "not-mobile" CSS styles, remove some`@media (max-width: 767.98px)` tricks 2. Use `CountFmt` for repo list, just like the repo header (and it matches GitHub, to avoid big numbers bloat the page) --- options/locale/locale_en-US.ini | 4 +--- templates/admin/repo/list.tmpl | 6 +++--- templates/base/head_navbar.tmpl | 10 +++++----- templates/explore/repo_list.tmpl | 17 ++++++++++++----- templates/repo/header.tmpl | 30 ++++++++++++++++-------------- web_src/css/base.css | 7 ------- web_src/css/helpers.css | 12 ++++++++++++ web_src/css/modules/navbar.css | 7 ------- web_src/css/repo/header.css | 23 ++--------------------- 9 files changed, 51 insertions(+), 65 deletions(-) diff --git a/options/locale/locale_en-US.ini b/options/locale/locale_en-US.ini index 08b87134f..3383bc0bc 100644 --- a/options/locale/locale_en-US.ini +++ b/options/locale/locale_en-US.ini @@ -1063,6 +1063,7 @@ watchers = Watchers stargazers = Stargazers stars_remove_warning = This will remove all stars from this repository. forks = Forks +stars = Stars reactions_more = and %d more unit_disabled = The site administrator has disabled this repository section. language_other = Other @@ -2966,9 +2967,6 @@ repos.unadopted.no_more = No more unadopted repositories found repos.owner = Owner repos.name = Name repos.private = Private -repos.watches = Watches -repos.stars = Stars -repos.forks = Forks repos.issues = Issues repos.size = Size repos.lfs_size = LFS Size diff --git a/templates/admin/repo/list.tmpl b/templates/admin/repo/list.tmpl index e977c8307..4b27d87a4 100644 --- a/templates/admin/repo/list.tmpl +++ b/templates/admin/repo/list.tmpl @@ -19,13 +19,13 @@ {{ctx.Locale.Tr "admin.repos.name"}} {{SortArrow "alphabetically" "reversealphabetically" $.SortType false}} - {{ctx.Locale.Tr "admin.repos.watches"}} + {{ctx.Locale.Tr "repo.watchers"}} - {{ctx.Locale.Tr "admin.repos.stars"}} + {{ctx.Locale.Tr "repo.stars"}} {{SortArrow "moststars" "feweststars" $.SortType false}} - {{ctx.Locale.Tr "admin.repos.forks"}} + {{ctx.Locale.Tr "repo.forks"}} {{SortArrow "mostforks" "fewestforks" $.SortType false}} {{ctx.Locale.Tr "admin.repos.issues"}} diff --git a/templates/base/head_navbar.tmpl b/templates/base/head_navbar.tmpl index 4f48dc82c..ffa2ef844 100644 --- a/templates/base/head_navbar.tmpl +++ b/templates/base/head_navbar.tmpl @@ -11,7 +11,7 @@ -
+
{{if .IsSigned}}
@@ -58,7 +58,7 @@
{{ctx.AvatarUtils.Avatar .SignedUser 24 "gt-mr-2"}} - {{.SignedUser.Name}} + {{.SignedUser.Name}} {{svg "octicon-triangle-down"}}
@@ -80,7 +80,7 @@ {{svg "octicon-stopwatch"}}
- {{ctx.Locale.Tr "active_stopwatch"}} + {{ctx.Locale.Tr "active_stopwatch"}}
@@ -122,7 +122,7 @@ {{svg "octicon-plus"}} {{svg "octicon-triangle-down"}} - {{ctx.Locale.Tr "create_new"}} + {{ctx.Locale.Tr "create_new"}}
@@ -144,7 +144,7 @@
{{ctx.AvatarUtils.Avatar .SignedUser 24 "gt-mr-2"}} - {{.SignedUser.Name}} + {{.SignedUser.Name}} {{svg "octicon-triangle-down"}}
diff --git a/templates/explore/repo_list.tmpl b/templates/explore/repo_list.tmpl index 7de3df5be..4c0f93ed2 100644 --- a/templates/explore/repo_list.tmpl +++ b/templates/explore/repo_list.tmpl @@ -30,16 +30,23 @@ {{end}}
-
+
{{if .PrimaryLanguage}} - - {{.PrimaryLanguage.Language}} + + + {{.PrimaryLanguage.Language}} {{end}} {{if not $.DisableStars}} - {{svg "octicon-star" 16}}{{.NumStars}} + + {{svg "octicon-star" 16}} + {{CountFmt .NumStars}} + {{end}} - {{svg "octicon-git-branch" 16}}{{.NumForks}} + + {{svg "octicon-git-branch" 16}} + {{CountFmt .NumForks}} +
{{$description := .DescriptionHTML $.Context}} diff --git a/templates/repo/header.tmpl b/templates/repo/header.tmpl index 6e0a9985f..e4d39839f 100644 --- a/templates/repo/header.tmpl +++ b/templates/repo/header.tmpl @@ -3,29 +3,31 @@
-
{{template "repo/icon" .}}
+
+ {{template "repo/icon" .}} +
- {{.Owner.Name}}/ - {{.Name}}
+ {{.Owner.Name}}/{{.Name}} +
{{if .IsArchived}} - {{ctx.Locale.Tr "repo.desc.archived"}} -
{{svg "octicon-archive" 18}}
+ {{ctx.Locale.Tr "repo.desc.archived"}} +
{{svg "octicon-archive" 18}}
{{end}} {{if .IsPrivate}} - {{ctx.Locale.Tr "repo.desc.private"}} -
{{svg "octicon-lock" 18}}
+ {{ctx.Locale.Tr "repo.desc.private"}} +
{{svg "octicon-lock" 18}}
{{else}} {{if .Owner.Visibility.IsPrivate}} - {{ctx.Locale.Tr "repo.desc.internal"}} -
{{svg "octicon-shield-lock" 18}}
+ {{ctx.Locale.Tr "repo.desc.internal"}} +
{{svg "octicon-shield-lock" 18}}
{{end}} {{end}} {{if .IsTemplate}} - {{ctx.Locale.Tr "repo.desc.template"}} -
{{svg "octicon-repo-template" 18}}
+ {{ctx.Locale.Tr "repo.desc.template"}} +
{{svg "octicon-repo-template" 18}}
{{end}} {{if eq .ObjectFormatName "sha256"}} {{ctx.Locale.Tr "repo.desc.sha256"}} @@ -76,7 +78,7 @@ - {{svg "octicon-repo-forked"}}{{ctx.Locale.Tr "repo.fork"}} + {{svg "octicon-repo-forked"}}{{ctx.Locale.Tr "repo.fork"}}