From 9946d318174c2e31198282eca18da1e849e63cf5 Mon Sep 17 00:00:00 2001
From: silverwind <me@silverwind.io>
Date: Fri, 9 Sep 2022 19:04:54 +0200
Subject: [PATCH 1/2] Improve commit status icons

- Show them on hover/focus (tippy default) instead of click
- If there is only one status, add href to trigger element
- Increase tippy interactiveBorder, making it easier to keep interactive
  tooltips open with sloppy mouse movement
- Fix a overflow issue in the commit list
---
 templates/repo/commit_statuses.tmpl | 2 +-
 web_src/js/features/repo-commit.js  | 1 -
 web_src/js/modules/tippy.js         | 1 +
 web_src/less/_repository.less       | 5 +++++
 4 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/templates/repo/commit_statuses.tmpl b/templates/repo/commit_statuses.tmpl
index d45d8a1df69f6..d682709082416 100644
--- a/templates/repo/commit_statuses.tmpl
+++ b/templates/repo/commit_statuses.tmpl
@@ -1,4 +1,4 @@
-<a class="ui link commit-statuses-trigger vm">{{template "repo/commit_status" .Status}}</a>
+<a class="ui link commit-statuses-trigger vm"{{if eq (len .Statuses) 1}}{{$status := index .Statuses 0}}{{if $status.TargetURL}} href="{{$status.TargetURL}}"{{end}}{{end}}>{{template "repo/commit_status" .Status}}</a>
 <div class="ui commit-statuses-popup commit-statuses tippy-target">
 	<div class="ui relaxed list divided">
 		{{range .Statuses}}
diff --git a/web_src/js/features/repo-commit.js b/web_src/js/features/repo-commit.js
index 170284f1010eb..3aba85091124e 100644
--- a/web_src/js/features/repo-commit.js
+++ b/web_src/js/features/repo-commit.js
@@ -61,7 +61,6 @@ export function initCommitStatuses() {
     const top = $('.repository.file.list').length > 0 || $('.repository.diff').length > 0;
 
     createTippy(this, {
-      trigger: 'click',
       content: this.nextElementSibling,
       placement: top ? 'top-start' : 'bottom-start',
       interactive: true,
diff --git a/web_src/js/modules/tippy.js b/web_src/js/modules/tippy.js
index 7fc273d1cc3e8..35c0c31224aa2 100644
--- a/web_src/js/modules/tippy.js
+++ b/web_src/js/modules/tippy.js
@@ -6,6 +6,7 @@ export function createTippy(target, opts = {}) {
     placement: target.getAttribute('data-placement') || 'top-start',
     animation: false,
     allowHTML: false,
+    interactiveBorder: 30,
     maxWidth: 500, // increase over default 350px
     arrow: `<svg width="16" height="7"><path d="m0 7 8-7 8 7Z" class="tippy-svg-arrow-outer"/><path d="m0 8 8-7 8 7Z" class="tippy-svg-arrow-inner"/></svg>`,
     ...(opts?.role && {theme: opts.role}),
diff --git a/web_src/less/_repository.less b/web_src/less/_repository.less
index 473d6f9dc7dd6..219597190da20 100644
--- a/web_src/less/_repository.less
+++ b/web_src/less/_repository.less
@@ -2827,6 +2827,11 @@ tbody.commit-list {
   vertical-align: middle;
 }
 
+// in the commit list, messages can wrap so we can use inline
+.commit-list .message-wrapper {
+  display: inline;
+}
+
 @media @mediaSm {
   tr.commit-list {
     width: 100%;

From b935213d1b87ab750b5c9ea6ee28ce9ad87e77e0 Mon Sep 17 00:00:00 2001
From: silverwind <me@silverwind.io>
Date: Fri, 9 Sep 2022 21:21:22 +0200
Subject: [PATCH 2/2] enable ignoreAttributes

---
 web_src/js/modules/tippy.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/web_src/js/modules/tippy.js b/web_src/js/modules/tippy.js
index 35c0c31224aa2..045df6f0a023a 100644
--- a/web_src/js/modules/tippy.js
+++ b/web_src/js/modules/tippy.js
@@ -7,6 +7,7 @@ export function createTippy(target, opts = {}) {
     animation: false,
     allowHTML: false,
     interactiveBorder: 30,
+    ignoreAttributes: true,
     maxWidth: 500, // increase over default 350px
     arrow: `<svg width="16" height="7"><path d="m0 7 8-7 8 7Z" class="tippy-svg-arrow-outer"/><path d="m0 8 8-7 8 7Z" class="tippy-svg-arrow-inner"/></svg>`,
     ...(opts?.role && {theme: opts.role}),