Skip to content

Commit f24eba3

Browse files
silverwindclaude
authored andcommitted
Fix action run view style regressions from go-gitea#36883
- Fix `.left-list-header` unreadable on dark theme by using `--color-text-light-2` instead of `--color-grey` - Fix `.graph-metrics` bullet inheriting blue color - Improve summary block layout with horizontal flexbox - Add dot separator between status and duration Co-Authored-By: Claude (claude-opus-4-6) <[email protected]>
1 parent a6c7ecb commit f24eba3

3 files changed

Lines changed: 22 additions & 10 deletions

File tree

web_src/js/components/ActionRunSummaryView.vue

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,11 @@ onBeforeUnmount(() => {
3232
<div>
3333
<div class="action-run-summary-block">
3434
<p class="action-run-summary-trigger">
35-
{{ locale.triggeredVia.replace('%s', run.triggerEvent) }}
36-
&nbsp;&nbsp;<relative-time :datetime="runTriggeredAtIso" prefix=" "/>
35+
{{ locale.triggeredVia.replace('%s', run.triggerEvent) }}&nbsp;&nbsp;<relative-time :datetime="runTriggeredAtIso"/>
3736
</p>
3837
<p class="tw-mb-0">
3938
<ActionRunStatus :locale-status="locale.status[run.status]" :status="run.status" :size="16"/>
40-
<span class="tw-ml-2">{{ locale.status[run.status] }}</span>
41-
<span class="tw-ml-3">{{ locale.totalDuration }} {{ run.duration || '–' }}</span>
39+
<span class="tw-ml-2">{{ locale.status[run.status] }}</span>&nbsp;&nbsp;<span>{{ locale.totalDuration }} {{ run.duration || '–' }}</span>
4240
</p>
4341
</div>
4442
<WorkflowGraph
@@ -53,12 +51,25 @@ onBeforeUnmount(() => {
5351
</template>
5452
<style scoped>
5553
.action-run-summary-block {
54+
display: flex;
55+
justify-content: space-between;
56+
align-items: center;
5657
padding: 12px;
5758
border-bottom: 1px solid var(--color-secondary);
5859
}
5960
6061
.action-run-summary-trigger {
61-
margin-bottom: 6px;
62+
margin-bottom: 0;
6263
color: var(--color-text-light-2);
6364
}
65+
66+
@media (max-width: 767.98px) {
67+
.action-run-summary-block {
68+
flex-direction: column;
69+
align-items: flex-start;
70+
}
71+
.action-run-summary-trigger {
72+
margin-bottom: 6px;
73+
}
74+
}
6475
</style>

web_src/js/components/RepoActionView.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -235,8 +235,8 @@ async function deleteArtifact(name: string) {
235235
}
236236
237237
.left-list-header {
238-
font-size: 12px;
239-
color: var(--color-grey-light);
238+
font-size: 13px;
239+
color: var(--color-text-light-2);
240240
}
241241
242242
.job-artifacts-item {

web_src/js/components/WorkflowGraph.vue

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -558,9 +558,9 @@ function onNodeClick(job: JobNode, event: MouseEvent) {
558558
<h4 class="graph-title">Workflow Dependencies</h4>
559559
<div class="graph-stats">
560560
{{ jobs.length }} jobs • {{ edges.length }} dependencies
561-
<span class="graph-metrics">
562-
• {{ graphMetrics.successRate }} success
563-
</span>
561+
<template v-if="graphMetrics">
562+
<span class="graph-metrics">{{ graphMetrics.successRate }} success</span>
563+
</template>
564564
</div>
565565
<div class="flex-text-block">
566566
<button @click="zoomIn" class="ui compact tiny icon button" title="Zoom in">
@@ -683,6 +683,7 @@ function onNodeClick(job: JobNode, event: MouseEvent) {
683683
align-items: center;
684684
padding: 8px 14px;
685685
background: var(--color-box-header);
686+
border-bottom: 1px solid var(--color-secondary);
686687
gap: 20px;
687688
flex-wrap: wrap;
688689
}

0 commit comments

Comments
 (0)