Skip to content

Commit 9abb333

Browse files
schrothbngandreini
andauthored
feat(editor): Improve UI for highlighted data, tags and rating in executions (#15926)
Co-authored-by: Giulio Andreini <[email protected]>
1 parent 4c9198d commit 9abb333

File tree

9 files changed

+925
-306
lines changed

9 files changed

+925
-306
lines changed

packages/frontend/@n8n/design-system/src/components/N8nTag/Tag.vue

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,18 @@ withDefaults(defineProps<TagProps>(), {
1818

1919
<style lang="scss" module>
2020
.tag {
21+
display: flex;
22+
align-items: center;
23+
justify-content: center;
2124
min-width: max-content;
22-
padding: 1px var(--spacing-4xs);
23-
color: var(--color-text-base);
24-
background-color: var(--color-background-base);
25-
border-radius: var(--border-radius-base);
26-
font-size: var(--font-size-2xs);
27-
25+
height: var(--tag-height);
26+
padding: var(--tag-padding);
27+
line-height: var(--tag-line-height);
28+
color: var(--tag-text-color);
29+
background-color: var(--tag-background-color);
30+
border: 1px solid var(--tag-border-color);
31+
border-radius: var(--tag-border-radius);
32+
font-size: var(--tag-font-size);
2833
transition: background-color 0.3s ease;
2934
3035
&.clickable {

packages/frontend/@n8n/design-system/src/css/_tokens.scss

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,17 @@
224224
--color-line-break: var(--prim-gray-320);
225225
--color-code-line-break: var(--prim-color-secondary-tint-200);
226226

227+
// Tag
228+
--tag-height: 22px;
229+
--tag-padding: 0 var(--spacing-4xs);
230+
--tag-background-color: var(--prim-gray-40);
231+
--tag-border-color: var(--color-foreground-light);
232+
--tag-border-radius: var(--border-radius-base);
233+
--tag-text-color: var(--color-text-base);
234+
--tag-font-size: var(--font-size-2xs);
235+
--tag-font-weight: var(--font-weight-regular);
236+
--tag-line-height: 0;
237+
227238
// Variables
228239
--color-variables-usage-font: var(--color-success);
229240
--color-variables-usage-syntax-bg: var(--color-success-tint-2);

packages/frontend/@n8n/i18n/src/locales/en.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -710,6 +710,7 @@
710710
"error.pageNotFound": "Oops, couldn’t find that",
711711
"executions.ExecutionStatus": "Execution status",
712712
"executions.concurrency.docsLink": "https://docs.n8n.io/hosting/scaling/concurrency-control/",
713+
"executionDetails.additionalActions": "Additional Actions",
713714
"executionDetails.confirmMessage.confirmButtonText": "Yes, delete",
714715
"executionDetails.confirmMessage.headline": "Delete Execution?",
715716
"executionDetails.confirmMessage.message": "Are you sure that you want to delete the current execution?",

packages/frontend/editor-ui/src/components/executions/workflow/VoteButtons.vue

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,18 +17,18 @@ const onVoteClick = (vote: AnnotationVote) => {
1717
<template>
1818
<div :class="$style.ratingIcon">
1919
<n8n-icon-button
20-
:class="{ [$style.up]: vote === 'up' }"
20+
:class="[$style.icon, vote === 'up' && $style.up]"
2121
type="tertiary"
2222
text
23-
size="medium"
23+
size="small"
2424
icon="thumbs-up"
2525
@click="onVoteClick('up')"
2626
/>
2727
<n8n-icon-button
28-
:class="{ [$style.down]: vote === 'down' }"
28+
:class="[$style.icon, vote === 'down' && $style.down]"
2929
type="tertiary"
3030
text
31-
size="medium"
31+
size="small"
3232
icon="thumbs-down"
3333
@click="onVoteClick('down')"
3434
/>
@@ -40,6 +40,14 @@ const onVoteClick = (vote: AnnotationVote) => {
4040
display: flex;
4141
flex-direction: row;
4242
43+
.icon {
44+
color: var(--color-text-light);
45+
46+
&:not(.up):not(.down):hover {
47+
color: var(--color-primary);
48+
}
49+
}
50+
4351
.up {
4452
color: var(--color-success);
4553
}

0 commit comments

Comments
 (0)