From 58abd4f06cb4f60777366122bc75a80fba074546 Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Sat, 9 Sep 2023 20:23:57 +0800 Subject: [PATCH] Improve issue list layout (#26983) Align everything with a new layout. * Use "baseline" for some special elements, the "flex-item-icon" is for the issue list only at the moment and I think it should be general enough now (but not using "flex-item-leading" anymore in this case). * Make the labels stretch themselves. --- templates/shared/issuelist.tmpl | 12 +++++------- web_src/css/repo/issue-list.css | 16 ++++++++++++++++ web_src/css/shared/flex-list.css | 9 +++++---- 3 files changed, 26 insertions(+), 11 deletions(-) diff --git a/templates/shared/issuelist.tmpl b/templates/shared/issuelist.tmpl index 8e91a58d51..80d4e3dd85 100644 --- a/templates/shared/issuelist.tmpl +++ b/templates/shared/issuelist.tmpl @@ -2,16 +2,14 @@ {{$approvalCounts := .ApprovalCounts}} {{range .Issues}}
-
+ +
{{if $.CanWriteIssuesOrPulls}} -
- -
+ {{end}} -
- {{template "shared/issueicon" .}} -
+ {{template "shared/issueicon" .}}
+
diff --git a/web_src/css/repo/issue-list.css b/web_src/css/repo/issue-list.css index f76b690cb1..17ae6ea38f 100644 --- a/web_src/css/repo/issue-list.css +++ b/web_src/css/repo/issue-list.css @@ -33,6 +33,22 @@ } } +#issue-list .flex-item-title .labels-list { + display: flex; + gap: 0.25em; +} + +#issue-list .flex-item-title .labels-list a { + display: flex; + text-decoration: none; +} + +#issue-list .flex-item-title .labels-list .label { + padding: 0 6px; + margin: 0; + min-height: 20px; +} + #issue-list .flex-item-body .branches { display: inline-flex; } diff --git a/web_src/css/shared/flex-list.css b/web_src/css/shared/flex-list.css index b5ae5cbd49..a7cfc8f163 100644 --- a/web_src/css/shared/flex-list.css +++ b/web_src/css/shared/flex-list.css @@ -34,9 +34,11 @@ } .flex-item .flex-item-icon { - display: flex; - align-items: center; - height: 20px; /* match the default flex-item-title height */ + align-self: baseline; /* mainly used by the issue list, to align the leading icon with the title */ +} + +.flex-item .flex-item-icon + .flex-item-main { + align-self: baseline; } .flex-item .flex-item-trailing { @@ -59,7 +61,6 @@ font-weight: var(--font-weight-semibold); word-break: break-word; min-width: 0; - min-height: 20px; } .flex-item .flex-item-title a {