Skip to content

Review fixes and enhancements #24526

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 22 commits into from
May 10, 2023
Merged

Review fixes and enhancements #24526

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
268f3ed
Review fixes and enhancements
silverwind May 4, 2023
7c7aa59
add avatars to review and review dismiss events
silverwind May 4, 2023
5438a5b
add missing inline avatars, fix avatar offset, icon size tweaks
silverwind May 4, 2023
043b6f8
Update web_src/css/repository.css
silverwind May 4, 2023
7686722
move to .Review.HTMLTypeColorName
silverwind May 8, 2023
e4c8f4a
Merge branch 'main' into fix-margin
silverwind May 8, 2023
293d98c
misc tweaks, decrease review list avatar size
silverwind May 8, 2023
9a72ff8
make fmt
silverwind May 8, 2023
ebbfa0e
final avatar fixes for review timeline
silverwind May 8, 2023
396b9d1
restore inline avatar
silverwind May 8, 2023
ccaa8b0
Merge branch 'main' into fix-margin
silverwind May 9, 2023
a963dd2
use constants
silverwind May 9, 2023
dbea7c3
add comment
silverwind May 9, 2023
38ee4da
use 16px everywhere except dismiss
silverwind May 9, 2023
dde032c
remove empty
silverwind May 9, 2023
ffa0443
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
d86e371
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
26d2a13
fix lint
silverwind May 10, 2023
b6cea6c
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
bcfc9b3
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
0724613
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
0cb87a8
Merge branch 'main' into fix-margin
GiteaBot May 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions modules/templates/helper.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,7 @@ func NewFuncMap() template.FuncMap {

"FilenameIsImage": FilenameIsImage,
"TabSizeClass": TabSizeClass,
"ReviewTypeColor": ReviewTypeColor,
}
}

Expand Down
15 changes: 15 additions & 0 deletions modules/templates/util_misc.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import (
"time"

activities_model "code.gitea.io/gitea/models/activities"
issue_model "code.gitea.io/gitea/models/issues"
repo_model "code.gitea.io/gitea/models/repo"
"code.gitea.io/gitea/modules/git"
giturl "code.gitea.io/gitea/modules/git/url"
Expand Down Expand Up @@ -135,6 +136,20 @@ func DiffLineTypeToStr(diffType int) string {
return "same"
}

func ReviewTypeColor(review issue_model.Review) string {
switch review.Type {
case 1:
return "green"
case 2:
return "grey"
case 3:
return "red"
case 4:
return "yellow"
}
return "grey"
}

// MigrationIcon returns a SVG name matching the service an issue/comment was migrated from
func MigrationIcon(hostname string) string {
switch hostname {
Expand Down
16 changes: 14 additions & 2 deletions templates/repo/issue/view_content/comments.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -372,11 +372,12 @@
<div class="timeline-item event">
{{if .OriginalAuthor}}
{{else}}
<a class="timeline-avatar"{{if gt .Poster.ID 0}} href="{{.Poster.HomeLink}}"{{end}}>
<a class="timeline-avatar{{if or (eq .Review.Type 2) (eq .Review.Type 3)}} timeline-avatar-offset{{end}}"{{if gt .Poster.ID 0}} href="{{.Poster.HomeLink}}"{{end}}>
{{avatar $.Context .Poster}}
</a>
{{end}}
<span class="badge{{if eq .Review.Type 1}} gt-bg-green gt-text-white{{else if eq .Review.Type 3}} gt-bg-red gt-text-white{{end}}">{{svg (printf "octicon-%s" .Review.Type.Icon)}}</span>
{{template "shared/user/avatarlink" dict "Context" $.Context "user" .Poster}}
<span class="text grey muted-links">
{{if .OriginalAuthor}}
<span class="text black">
Expand Down Expand Up @@ -408,6 +409,11 @@
<div class="content comment-container">
<div class="ui top attached header comment-header gt-df gt-ac gt-sb">
<div class="comment-header-left gt-df gt-ac">
{{if gt .Poster.ID 0}}
<a class="inline-timeline-avatar" href="{{.Poster.HomeLink}}">
{{avatar $.Context .Poster}}
</a>
{{end}}
<span class="text grey muted-links">
{{if .OriginalAuthor}}
<span class="text black gt-bold">
Expand Down Expand Up @@ -763,6 +769,7 @@
<img src="{{.Poster.AvatarLink $.Context}}">
</a>
<span class="badge grey">{{svg "octicon-x" 16}}</span>
{{template "shared/user/avatarlink" dict "Context" $.Context "user" .Poster}}
<span class="text grey muted-links">
{{template "shared/user/authorlink" .Poster}}
{{$reviewerName := ""}}
Expand All @@ -777,7 +784,12 @@
{{if .Content}}
<div class="timeline-item comment">
<div class="content">
<div class="ui top attached header arrow-top">
<div class="ui top attached header comment-header-left gt-df gt-ac arrow-top">
{{if gt .Poster.ID 0}}
<a class="inline-timeline-avatar" href="{{.Poster.HomeLink}}">
{{avatar $.Context .Poster}}
</a>
{{end}}
<span class="text grey muted-links">
{{$.locale.Tr "action.review_dismissed_reason"}}
</span>
Expand Down
42 changes: 15 additions & 27 deletions templates/repo/issue/view_content/pull.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{{if or .PullReviewers .OriginalReviews}}
<div class="comment box">
<div class="comment box timeline-item gt-pt-3 gt-pb-0">
<div class="content">
<div class="ui segment">
<h4>{{$.locale.Tr "repo.issues.review.reviewers"}}</h4>
Expand All @@ -8,8 +8,11 @@
<div class="ui divider"></div>
<div class="review-item">
<div class="review-item-left">
<span class="gt-mr-3 text {{ReviewTypeColor .Review}}">
{{svg (printf "octicon-%s" .Review.Type.Icon) 18}}
</span>
{{if .User}}
<a href="{{.User.HomeLink}}">
<a href="{{.User.HomeLink}}" class="gt-mr-2">
{{avatar $.Context .User}}
</a>
{{end}}
Expand All @@ -34,13 +37,13 @@
</div>
<div class="review-item-right">
{{if .Review.Stale}}
<span class="ui type-icon text grey" data-tooltip-content="{{$.locale.Tr "repo.issues.is_stale"}}">
<span class="ui text grey" data-tooltip-content="{{$.locale.Tr "repo.issues.is_stale"}}">
{{svg "octicon-hourglass" 16 "icon"}}
</span>
{{end}}
{{if (and $.Permission.IsAdmin (or (eq .Review.Type 1) (eq .Review.Type 3)) (not $.Issue.IsClosed))}}
<a href="#" class="ui muted icon dismiss-review-btn" data-review-id="dismiss-review-{{.Review.ID}}" data-tooltip-content="{{$.locale.Tr "repo.issues.dismiss_review"}}">
{{svg "octicon-x" 16}}
{{svg "octicon-x" 20}}
</a>
<div class="ui small modal" id="dismiss-review-modal">
<div class="header">
Expand All @@ -65,19 +68,9 @@
</div>
</div>
{{end}}
<span class="type-icon text {{if eq .Review.Type 1}}green
{{- else if eq .Review.Type 2}}grey
{{- else if eq .Review.Type 3}}red
{{- else if eq .Review.Type 4}}yellow
{{else}}grey{{end}}">

{{if .CanChange}}
<a href="#" class="ui icon re-request-review {{if .Checked}}checked{{end}}" data-issue-id="{{$.Issue.ID}}" data-tooltip-content="{{if .Checked}} {{$.locale.Tr "repo.issues.remove_request_review"}} {{else}} {{$.locale.Tr "repo.issues.re_request_review"}} {{end}}" data-id="{{.ItemID}}" data-update-url="{{$.RepoLink}}/issues/request_review">
{{if .Checked}} {{svg "octicon-trash"}} {{else}} {{svg "octicon-sync"}} {{end}}
</a>
{{end}}
{{svg (printf "octicon-%s" .Review.Type.Icon)}}
</span>
{{if .CanChange}}
<a href="#" class="ui icon muted re-request-review{{if .Checked}} checked{{end}}" data-issue-id="{{$.Issue.ID}}" data-tooltip-content="{{if .Checked}}{{$.locale.Tr "repo.issues.remove_request_review"}}{{else}}{{$.locale.Tr "repo.issues.re_request_review"}}{{end}}" data-id="{{.ItemID}}" data-update-url="{{$.RepoLink}}/issues/request_review">{{if .Checked}}{{svg "octicon-trash"}}{{else}}{{svg "octicon-sync"}}{{end}}</a>
{{end}}
</div>
</div>
{{end}}
Expand All @@ -86,22 +79,17 @@
<div class="ui divider"></div>
<div class="review-item">
<div class="review-item-left">
<span class="gt-mr-3 text {{ReviewTypeColor .Review}}">
{{svg (printf "octicon-%s" .Type.Icon) 18}}
</span>
<a href="{{$.Repository.OriginalURL}}" data-tooltip-content="{{$.locale.Tr "repo.migrated_from_fake" ($.Repository.GetOriginalURLHostname|Escape) | Safe}}">
<span class="text black ">
<span class="text black">
{{svg (MigrationIcon $.Repository.GetOriginalURLHostname)}}
{{.OriginalAuthor}}
</span>
</a>
</div>
<div class="review-item-right">
<span class="type-icon text {{if eq .Type 1}}green
{{- else if eq .Type 2}}grey
{{- else if eq .Type 3}}red
{{- else if eq .Type 4}}yellow
{{else}}grey{{end}}">
{{svg (printf "octicon-%s" .Type.Icon)}}
</span>
</div>
<div class="review-item-right"></div>
</div>
{{end}}
</div>
Expand Down
20 changes: 5 additions & 15 deletions templates/repo/issue/view_content/sidebar.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,25 +54,15 @@
{{range .PullReviewers}}
<div class="item gt-mb-2">
{{if .User}}
<a class="muted sidebar-item-link" href="{{.User.HomeLink}}">
{{avatar $.Context .User 28 "gt-mr-3"}}
{{.User.GetDisplayName}}
</a>
<a class="muted sidebar-item-link" href="{{.User.HomeLink}}">{{avatar $.Context .User 28 "gt-mr-3"}}{{.User.GetDisplayName}}</a>
{{else if .Team}}
<span class="text">{{svg "octicon-people" 16 "teamavatar"}}{{$.Issue.Repo.OwnerName}}/{{.Team.Name}}</span>
{{end}}
<span class="ui right type-icon text {{if eq .Review.Type 1}}green
{{- else if eq .Review.Type 2}}grey
{{- else if eq .Review.Type 3}}red
{{- else if eq .Review.Type 4}}yellow
{{- else}}grey{{end}} right ">

<span class="ui right gt-df gt-ac gt-gap-3">
{{if .CanChange}}
<a href="#" class="ui icon re-request-review {{if .Checked}}checked{{end}}" data-tooltip-content="{{if .Checked}} {{$.locale.Tr "repo.issues.remove_request_review"}} {{else}} {{$.locale.Tr "repo.issues.re_request_review"}} {{end}}" data-issue-id="{{$.Issue.ID}}" data-id="{{.ItemID}}" data-update-url="{{$.RepoLink}}/issues/request_review">
{{if .Checked}} {{svg "octicon-trash"}} {{else}} {{svg "octicon-sync"}} {{end}}
</a>
<a href="#" class="ui muted icon re-request-review{{if .Checked}} checked{{end}}" data-tooltip-content="{{if .Checked}}{{$.locale.Tr "repo.issues.remove_request_review"}}{{else}}{{$.locale.Tr "repo.issues.re_request_review"}}{{end}}" data-issue-id="{{$.Issue.ID}}" data-id="{{.ItemID}}" data-update-url="{{$.RepoLink}}/issues/request_review">{{if .Checked}}{{svg "octicon-trash"}}{{else}}{{svg "octicon-sync"}}{{end}}</a>
{{end}}
{{svg (printf "octicon-%s" .Review.Type.Icon)}}
{{svg (printf "octicon-%s" .Review.Type.Icon) 18 (printf "text %s" (ReviewTypeColor .Review))}}
</span>
</div>
{{end}}
Expand All @@ -84,7 +74,7 @@
{{.OriginalAuthor}}
</span>
</a>
<span class="ui right type-icon text {{if eq .Type 1}}green
<span class="ui right text {{if eq .Type 1}}green
{{- else if eq .Type 2}}grey
{{- else if eq .Type 3}}red
{{- else if eq .Type 4}}yellow
Expand Down
36 changes: 17 additions & 19 deletions web_src/css/repository.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -691,49 +691,41 @@
margin-right: 5px;
}

.repository.view.issue .pull .merge.box .timeline-avatar {
.repository.view.issue .merge.box .timeline-avatar {
margin-top: 3px;
margin-left: 4px;
}

.repository.view.issue .pull .merge.box .branch-update.grid .row {
.repository.view.issue .merge.box .branch-update.grid .row {
padding-bottom: 1rem;
}

.repository.view.issue .pull .merge.box .branch-update.grid .row .icon {
.repository.view.issue .merge.box .branch-update.grid .row .icon {
margin-top: 1.1rem;
}

.repository.view.issue .pull .review-item {
.repository.view.issue .review-item {
display: flex;
justify-content: space-between;
align-items: center;
}

.repository.view.issue .pull .review-item .review-item-left,
.repository.view.issue .pull .review-item .review-item-right {
.review-item-left,
.review-item-right {
display: flex;
align-items: center;
}

.repository.view.issue .pull .review-item .text {
margin: 0.3em 0 0.5em 0.5em;
}

.repository.view.issue .pull .review-item .type-icon {
align-self: flex-start;
margin-right: 1em;
}

.repository.view.issue .pull .review-item .type-icon i {
line-height: 1.8em;
.review-item-right {
gap: 8px;
margin-left: 4px;
}

.repository.view.issue .pull .review-item .divider {
.repository.view.issue .review-item .divider {
margin: 0.5rem 0;
}

.repository.view.issue .pull .review-item .review-content {
.repository.view.issue .review-item .review-content {
padding: 1em 0 1em 3.8em;
}

Expand Down Expand Up @@ -782,6 +774,10 @@
padding-bottom: 8px;
}

.repository.view.issue .comment-list .timeline-item-group .timeline-avatar-offset {
margin-top: 48px;
}

.repository.view.issue .comment-list .timeline-item {
margin-left: 16px;
position: relative;
Expand Down Expand Up @@ -3261,6 +3257,8 @@ td.blob-excerpt {
.repository.view.issue .comment-list .timeline .inline-timeline-avatar {
display: flex;
margin-bottom: auto;
margin-left: 6px;
margin-right: 2px;
}
.repository.view.issue .comment-list .timeline .inline-timeline-avatar img.avatar {
height: 24px;
Expand Down