improve

Author: yp05327

options/locale/locale_en-US.ini

@@ -1751,6 +1751,8 @@ pulls.status_checks_failure = Some checks failed
 pulls.status_checks_error = Some checks reported errors
 pulls.status_checks_requested = Required
 pulls.status_checks_details = Details
+pulls.status_checks_hide_all = Hide all checks
+pulls.status_checks_show_all = Show all checks
 pulls.update_branch = Update branch by merge
 pulls.update_branch_rebase = Update branch by rebase
 pulls.update_branch_success = Branch update was successful

templates/repo/pulls/status.tmpl

@@ -14,21 +14,29 @@
 			{{else}}
 				{{$.locale.Tr "repo.pulls.status_checking"}}
 			{{end}}
+			<div class="ui right">
+				<a class="hide-all-checks muted" 
+				data-show-all="{{$.locale.Tr "repo.pulls.status_checks_show_all"}}"
+				data-hide-all="{{$.locale.Tr "repo.pulls.status_checks_hide_all"}}">
+				{{$.locale.Tr "repo.pulls.status_checks_hide_all"}}</a>
 			</div>
+		</div>
 	{{end}}
 
-	{{range $.LatestCommitStatuses}}
-		<div class="ui attached segment pr-status">
-			{{template "repo/commit_status" .}}
-			<div class="status-context">
-				<span>{{.Context}} <span class="text grey">{{.Description}}</span></span>
-				<div class="ui status-details">
-					{{if $.is_context_required}}
-						{{if (call $.is_context_required .Context)}}<div class="ui label">{{$.locale.Tr "repo.pulls.status_checks_requested"}}</div>{{end}}
-					{{end}}
-					<span>{{if .TargetURL}}<a href="{{.TargetURL}}">{{$.locale.Tr "repo.pulls.status_checks_details"}}</a>{{end}}</span>
+	<div class="ui attached segment pr-status-list">
+		{{range $.LatestCommitStatuses}}
+			<div class="ui attached segment pr-status">
+				{{template "repo/commit_status" .}}
+				<div class="status-context">
+					<span>{{.Context}} <span class="text grey">{{.Description}}</span></span>
+					<div class="ui status-details">
+						{{if $.is_context_required}}
+							{{if (call $.is_context_required .Context)}}<div class="ui label">{{$.locale.Tr "repo.pulls.status_checks_requested"}}</div>{{end}}
+						{{end}}
+						<span>{{if .TargetURL}}<a href="{{.TargetURL}}">{{$.locale.Tr "repo.pulls.status_checks_details"}}</a>{{end}}</span>
+					</div>
 				</div>
 			</div>
-		</div>
-	{{end}}
+		{{end}}
+	</div>
 {{end}}

web_src/css/repo.css

@@ -3284,6 +3284,19 @@ tbody.commit-list {
   }
 }
 
+.ui.segment.pr-status-list {
+  padding: 0 0.1em;
+  max-height: 30vh;
+  overflow-x: auto;
+  transition: max-height 0.4s;
+}
+
+.ui.segment.pr-status-list.hide{
+  padding: 0 0.1em;
+  max-height: 0;
+  transition: max-height 0.4s;
+}
+
 .pr-status {
   padding: 0 !important; /* To clear fomantic's padding on .ui.segment elements */
   display: flex;

web_src/js/features/repo-issue-pr-status.js

@@ -0,0 +1,17 @@
+import $ from 'jquery';
+
+export function initRepoPullRequestCommitStatus() {
+  const $prStatusList = $('.pr-status-list');
+
+  $('.hide-all-checks').on('click', async (e) => {
+    e.preventDefault();
+    const $this = $(e.currentTarget);
+    if ($prStatusList.hasClass('hide')) {
+      $prStatusList.removeClass('hide');
+      $this.text($this.attr('data-hide-all'));
+    } else {
+      $prStatusList.addClass('hide');
+      $this.text($this.attr('data-show-all'));
+    }
+  });
+}

web_src/js/features/repo-legacy.js

@@ -21,6 +21,7 @@ import {initCommentContent, initMarkupContent} from '../markup/content.js';
 import {initCompReactionSelector} from './comp/ReactionSelector.js';
 import {initRepoSettingBranches} from './repo-settings.js';
 import {initRepoPullRequestMergeForm} from './repo-issue-pr-form.js';
+import {initRepoPullRequestCommitStatus} from './repo-issue-pr-status.js';
 import {hideElem, showElem} from '../utils/dom.js';
 import {getComboMarkdownEditor, initComboMarkdownEditor} from './comp/ComboMarkdownEditor.js';
 import {attachRefIssueContextPopup} from './contextpopup.js';
@@ -547,6 +548,7 @@ export function initRepository() {
     initCompReactionSelector($(document));
 
     initRepoPullRequestMergeForm();
+    initRepoPullRequestCommitStatus();
   }
 
   // Pull request