fine tune and add comments

Author: wxiaoguang

routers/web/repo/issue.go

@@ -556,7 +556,7 @@ func renderMilestones(ctx *context.Context) {
 	ctx.Data["ClosedMilestones"] = closedMilestones
 }
 
-// RetrieveRepoMilestonesAndAssignees find all the milestones, assignees, and reviewers of a repository
+// RetrieveRepoMilestonesAndAssignees find all the milestones and assignees of a repository
 func RetrieveRepoMilestonesAndAssignees(ctx *context.Context, repo *repo_model.Repository) {
 	var err error
 	ctx.Data["OpenMilestones"], err = db.Find[issues_model.Milestone](ctx, issues_model.FindMilestoneOptions{
@@ -676,7 +676,7 @@ type issueSidebarReviewersData struct {
 	CurrentPullReviewers []*repoReviewerSelection
 }
 
-// RetrieveRepoReviewers find all reviewers of a repository
+// RetrieveRepoReviewers find all reviewers of a repository. If issue is nil, it means the doer is creating a new PR.
 func RetrieveRepoReviewers(ctx *context.Context, repo *repo_model.Repository, issue *issues_model.Issue, canChooseReviewer bool) {
 	data := &issueSidebarReviewersData{}
 	data.RepoLink = ctx.Repo.RepoLink

templates/repo/issue/sidebar/reviewer_list.tmpl

@@ -37,7 +37,7 @@
 		</div>
 	</div>
 
-	<div class="ui list flex-items-block">
+	<div class="ui relaxed list flex-items-block">
 		<span class="item empty-list {{if or $data.OriginalReviews $data.CurrentPullReviewers}}tw-hidden{{end}}">
 			{{ctx.Locale.Tr "repo.issues.new.no_reviewers"}}
 		</span>
@@ -81,7 +81,7 @@
 				</div>
 			</div>
 		{{end}}
-		{{range .OriginalReviews}}
+		{{range $data.OriginalReviews}}
 			<div class="item">
 				<div class="flex-text-inline tw-flex-1">
 					{{$originalURLHostname := $data.Repository.GetOriginalURLHostname}}

web_src/css/repo.css

@@ -53,6 +53,7 @@
 .issue-sidebar-combo .ui.dropdown .item:not(.checked) svg.octicon-check {
   visibility: hidden;
 }
+/* ideally, we should move these styles to ".ui.dropdown .menu.flex-items-menu > .item ...", could be done later */
 .issue-sidebar-combo .ui.dropdown .menu > .item > img,
 .issue-sidebar-combo .ui.dropdown .menu > .item > svg {
   margin: 0;

web_src/js/features/repo-issue-sidebar-combolist.ts

@@ -80,9 +80,9 @@ export function initIssueSidebarComboList(container: HTMLElement) {
     async onHide() {
       const changedValues = collectCheckedValues(elDropdown);
       if (updateUrl) {
-        await updateToBackend(changedValues);
+        await updateToBackend(changedValues); // send requests to backend and reload the page
       } else {
-        syncList(changedValues);
+        syncList(changedValues); // only update the list in the sidebar
       }
     },
   });