Skip to content

Commit 9e028d8

Browse files
Fix comment header padding (#33377)
1 parent 3e1b63f commit 9e028d8

File tree

5 files changed

+10
-14
lines changed

5 files changed

+10
-14
lines changed

templates/repo/diff/comments.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
{{template "shared/user/avatarlink" dict "user" .Poster}}
99
{{end}}
1010
<div class="content comment-container">
11-
<div class="ui top attached header comment-header tw-flex tw-items-center tw-justify-between">
11+
<div class="comment-header">
1212
<div class="comment-header-left tw-flex tw-items-center">
1313
{{if .OriginalAuthor}}
1414
<span class="text black tw-font-semibold tw-mr-1">

templates/repo/issue/view_content.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
</a>
1414
{{end}}
1515
<div class="content comment-container">
16-
<div class="ui top attached header comment-header tw-flex tw-items-center tw-justify-between" role="heading" aria-level="3">
16+
<div class="comment-header" role="heading" aria-level="3">
1717
<div class="comment-header-left tw-flex tw-items-center">
1818
{{if .Issue.OriginalAuthor}}
1919
<span class="text black tw-font-semibold">

templates/repo/issue/view_content/comments.tmpl

+5-5
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
</a>
2727
{{end}}
2828
<div class="content comment-container">
29-
<div class="ui top attached header comment-header tw-flex tw-items-center tw-justify-between" role="heading" aria-level="3">
29+
<div class="comment-header" role="heading" aria-level="3">
3030
<div class="comment-header-left tw-flex tw-items-center">
3131
{{if .OriginalAuthor}}
3232
<span class="text black tw-font-semibold tw-mr-1">
@@ -393,7 +393,7 @@
393393
{{if or .Content .Attachments}}
394394
<div class="timeline-item comment">
395395
<div class="content comment-container">
396-
<div class="ui top attached header comment-header tw-flex tw-items-center tw-justify-between">
396+
<div class="comment-header">
397397
<div class="comment-header-left tw-flex tw-items-center">
398398
{{if gt .Poster.ID 0}}
399399
<a class="inline-timeline-avatar" href="{{.Poster.HomeLink}}">
@@ -633,8 +633,8 @@
633633
</div>
634634
{{if .Content}}
635635
<div class="timeline-item comment">
636-
<div class="content">
637-
<div class="ui top attached header comment-header-left tw-flex tw-items-center arrow-top">
636+
<div class="content comment-container">
637+
<div class="comment-header arrow-top">
638638
{{if gt .Poster.ID 0}}
639639
<a class="inline-timeline-avatar" href="{{.Poster.HomeLink}}">
640640
{{ctx.AvatarUtils.Avatar .Poster 24}}
@@ -644,7 +644,7 @@
644644
{{ctx.Locale.Tr "action.review_dismissed_reason"}}
645645
</span>
646646
</div>
647-
<div class="ui attached segment">
647+
<div class="ui attached segment comment-body">
648648
<div class="render-content markup">
649649
{{if .RenderedContent}}
650650
{{.RenderedContent}}

templates/repo/issue/view_content/conversation.tmpl

+1-1
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
{{$createdSubStr:= DateUtils.TimeSince .CreatedUnix}}
5858
<div class="comment code-comment" id="{{.HashTag}}">
5959
<div class="content comment-container">
60-
<div class="header comment-header">
60+
<div class="comment-header">
6161
<div class="comment-header-left tw-flex tw-items-center">
6262
{{if not .OriginalAuthor}}
6363
<a class="avatar">

web_src/css/repo.css

+2-6
Original file line numberDiff line numberDiff line change
@@ -1480,16 +1480,12 @@ td .commit-summary {
14801480
}
14811481

14821482
.comment-header {
1483-
border: none !important;
14841483
background: var(--color-box-header);
1485-
border-bottom: 1px solid var(--color-secondary) !important;
1486-
font-weight: var(--font-weight-normal) !important;
1487-
padding: 0.5rem 1rem;
1488-
margin: 0 !important;
1484+
border-bottom: 1px solid var(--color-secondary);
1485+
padding: 0 1rem;
14891486
position: relative;
14901487
color: var(--color-text);
14911488
min-height: 41px;
1492-
background-color: var(--color-box-header);
14931489
display: flex;
14941490
justify-content: space-between;
14951491
align-items: center;

0 commit comments

Comments
 (0)