@andrian.yablonskyy/thub-coordinator 1.0.21 → 1.0.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andrian.yablonskyy/thub-coordinator",
-  "version": "1.0.21",
+  "version": "1.0.22",
   "description": "TestHub Coordinator — job queue, resource registry, scheduler, heartbeat monitor, log/artifact store and web dashboard",
   "main": "src/server.js",
   "engines": {

package/public/css/thub.css

@@ -28,3 +28,28 @@
   from { transform: scaleX(1); }
   to { transform: scaleX(0); }
 }
+
+/* Hints (Bootstrap tooltips, public/js/tooltips.js): left-aligned — many
+   are multi-line sentences — and 0.9 of Bootstrap's .875rem font size. */
+.tooltip {
+  --bs-tooltip-font-size: calc(0.875rem * 0.9);
+}
+
+.tooltip-inner {
+  text-align: left;
+}
+
+/* Resource card (views/mixins/resource.pug): 30% wider than Bootstrap's
+   default 500px modal. Only applies from 576px up, like Bootstrap's own
+   width — below that the modal fills the screen as before. */
+.modal.thub-resource-card {
+  --bs-modal-width: calc(500px * 1.3);
+}
+
+/* A status badge inside a page heading (e.g. the job page's state): badges
+   are 0.75em of their parent, which in an h4 makes them oversized — keep
+   it close to the badges in tables, centered on the heading text. */
+.thub-heading-badge {
+  font-size: 0.55em;
+  vertical-align: middle;
+}

package/src/web/routes.js

@@ -408,7 +408,7 @@ function createWebRouter({ services, config }){
     }
 
     parts.push(s.coordinatorUpdate
-      ? `Coordinator v${s.coordinatorVersion} → v${s.coordinatorUpdate} available (Update app button).`
+      ? `Coordinator v${s.coordinatorVersion} → v${s.coordinatorUpdate} available.`
       : `Coordinator is up to date (v${s.coordinatorVersion}).`);
 
     const connected = services.registry.list().filter((r) => r.status !== RESOURCE_STATES.OUT_OF_SERVICE && r.last_heartbeat_at),
@@ -435,7 +435,7 @@ function createWebRouter({ services, config }){
       parts.push(`Some checks failed: ${s.error}`);
     }
     const behind = s.coordinatorUpdate || outdated.length || unknown.length;
-    return { type: behind || s.error ? 'warning' : 'success', text: parts.join(' ') };
+    return { type: behind || s.error ? 'warning' : 'success', text: parts.join('\n') };
   }));
 
   router.post('/updates/coordinator', requireAdminRole, updateAction(async (req) => {

package/views/jobs/list.pug

@@ -48,7 +48,7 @@ block content
           td: +jobBadge(j.state)
           td= j.resource ? j.resource.name : j.resource_name ? `${j.resource_name} (removed)` : '—'
           td= fmtDate(j.created_at)
-          td(title=j.finished_at ? `Finished ${fmtDate(j.finished_at)}` : null)= j.duration_sec != null ? fmtDuration(j.duration_sec * 1000) : '—'
+          td(data-bs-toggle=j.finished_at ? 'tooltip' : null data-bs-title=j.finished_at ? `Finished ${fmtDate(j.finished_at)}` : null)= j.duration_sec != null ? fmtDuration(j.duration_sec * 1000) : '—'
           if canManage
             td.text-end
               if j.active
@@ -59,7 +59,7 @@ block content
                   onsubmit="return confirm(this.dataset.confirm)"
                 )
                   input(type="hidden" name="returnTo" value=returnTo)
-                  button.btn.btn-sm.btn-outline-danger(type="submit" aria-label="Cancel" title="Cancel this job")
+                  button.btn.btn-sm.btn-outline-danger(type="submit" aria-label="Cancel" data-bs-toggle="tooltip" data-bs-title="Cancel this job")
                     i.bi.bi-x-circle
 
   if canManage

package/views/jobs/show.pug

@@ -7,7 +7,7 @@ block content
     h1.h4.mb-0
       = job.id
       |
-      +jobBadge(job.state)
+      +jobBadge(job.state)(class="thub-heading-badge ms-1")
     if canCancel
       form(method="post" action=`/jobs/${job.id}/cancel` onsubmit=`return confirm('Cancel job ${job.id}?')`)
         button.btn.btn-outline-danger.btn-sm(type="submit") Cancel
@@ -29,7 +29,7 @@ block content
     dd.col-sm-10= fmtDate(job.created_at)
     if job.duration_sec != null
       dt.col-sm-2 Duration
-      dd.col-sm-10(title=`Finished ${fmtDate(job.finished_at)}`)= fmtDuration(job.duration_sec * 1000)
+      dd.col-sm-10(data-bs-toggle="tooltip" data-bs-title=`Finished ${fmtDate(job.finished_at)}`)= fmtDuration(job.duration_sec * 1000)
     if job.message
       dt.col-sm-2 Message
       dd.col-sm-10= job.message

package/views/mixins/resource.pug

@@ -226,7 +226,7 @@ mixin resourceCard(r, groupsById, returnTo)
   - const isLoopback = (a) => /^127\./.test(a) || a === '::1' || a === 'localhost'
   - const local = ((r.host_info && r.host_info.addresses) || []).filter((a) => !isLoopback(a.address))
   - const external = r.remote_addr && !isLoopback(r.remote_addr) ? r.remote_addr : null
-  .modal.fade(id=`resource-card-${r.id}` tabindex="-1" aria-labelledby=`resource-card-${r.id}-title` aria-hidden="true")
+  .modal.fade.thub-resource-card(id=`resource-card-${r.id}` tabindex="-1" aria-labelledby=`resource-card-${r.id}-title` aria-hidden="true")
     .modal-dialog.modal-dialog-centered
       .modal-content
         .modal-header
@@ -279,7 +279,7 @@ mixin resourceCard(r, groupsById, returnTo)
             dd.col-sm-8
               if r.group_ids.length
                 each gid in r.group_ids
-                  span.badge.text-bg-info.me-1(title=gid)= groupsById[gid] ? groupsById[gid].name : `(unknown: ${gid})`
+                  span.badge.text-bg-info.me-1(data-bs-toggle="tooltip" data-bs-title=gid)= groupsById[gid] ? groupsById[gid].name : `(unknown: ${gid})`
               else
                 span.text-body-secondary —
             dt.col-sm-4 Busy source / reason

package/views/mixins/status-badge.pug

@@ -11,4 +11,4 @@ mixin resourceBadge(status, removing)
 
 mixin jobBadge(state)
   - const map = { QUEUED: 'secondary', ASSIGNED: 'info', PREPARING: 'info', RUNNING: 'primary', PASSED: 'success', FAILED: 'danger', ERROR: 'danger', TIMEOUT: 'warning', CANCELED: 'secondary', LOST: 'danger' }
-  span.badge(class=`text-bg-${map[state] || 'secondary'}`)= state
+  span.badge(class=`text-bg-${map[state] || 'secondary'}`)&attributes(attributes)= state

package/views/resources/list.pug

@@ -42,7 +42,7 @@ block content
           td
             if r.group_ids.length
               each gid in r.group_ids
-                span.badge.text-bg-info.me-1(title=gid)= groupsById[gid] ? groupsById[gid].name : `(unknown: ${gid})`
+                span.badge.text-bg-info.me-1(data-bs-toggle="tooltip" data-bs-title=gid)= groupsById[gid] ? groupsById[gid].name : `(unknown: ${gid})`
             else
               span.text-body-secondary —
           td= r.busy_source ? `${r.busy_source}${r.busy_reason ? ' — ' + r.busy_reason : ''}` : '—'