diff --git a/templates/repo/diff/comment_form.tmpl b/templates/repo/diff/comment_form.tmpl index c82d32c214d8..489336a59fbb 100644 --- a/templates/repo/diff/comment_form.tmpl +++ b/templates/repo/diff/comment_form.tmpl @@ -1,9 +1,4 @@ {{if and $.root.SignedUserID (not $.Repository.IsArchived)}} - {{if $.hidden}} - - {{end}}
{{$.root.CsrfTokenHtml}} @@ -26,7 +21,7 @@ {{.i18n.Tr "loading"}} - {{end}} -
+
{{range $comms}} {{ $createdSubStr:= TimeSinceUnix .CreatedUnix $.Lang }} -
+
@@ -572,21 +572,33 @@
{{end}}
- {{template "repo/diff/comment_form_datahandler" dict "hidden" true "reply" (index $comms 0).ReviewID "root" $ "comment" (index $comms 0)}} - - {{if and $.CanMarkConversation $isNotPending}} - - {{end}} - - {{if $resolved}} - {{$resolveDoer.Name}} {{$.i18n.Tr "repo.issues.review.resolved_by"}} - {{end}} +
+
+ {{if and $.CanMarkConversation $isNotPending}} + + {{end}} + {{if and $.SignedUserID (not $.Repository.IsArchived)}} + + {{end}} +
+
+ {{template "repo/diff/comment_form_datahandler" dict "hidden" true "reply" (index $comms 0).ReviewID "root" $ "comment" (index $comms 0)}}
{{end}} diff --git a/web_src/js/index.js b/web_src/js/index.js index 2fce21b8d3c3..cce2fd6caa85 100644 --- a/web_src/js/index.js +++ b/web_src/js/index.js @@ -1175,7 +1175,7 @@ async function initRepository() { const form = $(e.currentTarget).closest('form'); if (form.length > 0 && form.hasClass('comment-form')) { form.addClass('hide'); - form.parent().find('button.comment-form-reply').show(); + form.closest('.comment-code-cloud').find('button.comment-form-reply').show(); } else { form.closest('.comment-code-cloud').remove(); } @@ -1333,7 +1333,7 @@ function initPullRequestReview() { $(document).on('click', 'button.comment-form-reply', function (e) { e.preventDefault(); $(this).hide(); - const form = $(this).parent().find('.comment-form'); + const form = $(this).closest('.comment-code-cloud').find('.comment-form'); form.removeClass('hide'); const $textarea = form.find('textarea'); let $simplemde; diff --git a/web_src/less/_base.less b/web_src/less/_base.less index 7e563d2f2332..00aea43683cd 100644 --- a/web_src/less/_base.less +++ b/web_src/less/_base.less @@ -469,6 +469,10 @@ a.ui.card:hover, border-top-color: var(--color-secondary-light-1) !important; } +.ui.comments .comment .text { + margin: 0; +} + .ui.comments .comment .text, .ui.comments .comment .author { color: var(--color-text); @@ -1633,6 +1637,11 @@ a.ui.label:hover { border-color: var(--color-light-border); } +.ui.labeled.icon.buttons > .button > .icon, +.ui.labeled.icon.button > .icon { + background: var(--color-hover); +} + .ui.blue.button, .ui.blue.buttons .button, .ui.primary.button, diff --git a/web_src/less/_repository.less b/web_src/less/_repository.less index 5ff51b1d6df6..cf9784689c01 100644 --- a/web_src/less/_repository.less +++ b/web_src/less/_repository.less @@ -1058,7 +1058,7 @@ .code-comment { border: 1px solid transparent; - padding: .5rem 0; + padding: .25rem .5rem; margin: 0; .content { @@ -1076,6 +1076,10 @@ } } + .comment-content { + margin-left: 36px; + } + .avatar.image { width: 28px; height: 28px; @@ -1094,7 +1098,7 @@ } button.comment-form-reply { - margin-left: 0; + margin: 0; } } diff --git a/web_src/less/_review.less b/web_src/less/_review.less index f6f5df20a21e..f329ab5c90cf 100644 --- a/web_src/less/_review.less +++ b/web_src/less/_review.less @@ -112,14 +112,10 @@ display: block; } } +} - button.comment-form-reply { - margin: .5em .5em .5em 4.5em; - } - - form.comment-form-reply { - margin: 0 0 0 1em; - } +.diff-file-body .comment-form { + margin: 0 0 0 3em; } .file-comment {