Skip to content

Commit 6cd682a

Browse files
DrJKLgithub-actions
authored andcommitted
[backport 1.27] fix: Status indicator and close button appearing together (#5738)
Small fix for the close/status visibility overlap <img width="392" height="128" alt="image" src="https://github.com/user-attachments/assets/af25f1d7-a8c3-4155-9123-9fa10724e8db" /> ![20250923-2140-26 4569642](https://github.com/user-attachments/assets/e1b00a3f-d6e9-416b-9014-df0f9241082e) ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-5738-fix-Status-indicator-and-close-button-appearing-together-2776d73d3650813e9601e519c8a85043) by [Unito](https://www.unito.io) --------- Co-authored-by: github-actions <github-actions@github.com>
1 parent 77ca6d5 commit 6cd682a

File tree

2 files changed

+7
-23
lines changed

2 files changed

+7
-23
lines changed

src/components/topbar/WorkflowTab.vue

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<template>
22
<div
33
ref="workflowTabRef"
4-
class="flex p-2 gap-2 workflow-tab"
4+
class="flex p-2 gap-2 workflow-tab group"
55
v-bind="$attrs"
66
@mouseenter="handleMouseEnter"
77
@mouseleave="handleMouseLeave"
@@ -11,9 +11,13 @@
1111
{{ workflowOption.workflow.filename }}
1212
</span>
1313
<div class="relative">
14-
<span v-if="shouldShowStatusIndicator" class="status-indicator">•</span>
14+
<span
15+
v-if="shouldShowStatusIndicator"
16+
class="group-hover:hidden absolute font-bold text-2xl top-1/2 left-1/2 -translate-1/2 z-10 bg-(--comfy-menu-secondary-bg) w-4"
17+
>•</span
18+
>
1519
<Button
16-
class="close-button p-0 w-auto"
20+
class="close-button p-0 w-auto invisible"
1721
icon="pi pi-times"
1822
text
1923
severity="secondary"
@@ -174,18 +178,6 @@ onUnmounted(() => {
174178
})
175179
</script>
176180

177-
<style scoped>
178-
@reference '../../assets/css/style.css';
179-
180-
.status-indicator {
181-
@apply absolute font-bold;
182-
font-size: 1.5rem;
183-
top: 50%;
184-
left: 50%;
185-
transform: translate(-50%, -50%);
186-
}
187-
</style>
188-
189181
<style>
190182
.p-tooltip.workflow-tab-tooltip {
191183
z-index: 1200 !important;

src/components/topbar/WorkflowTabs.vue

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -360,14 +360,6 @@ onUpdated(() => {
360360
@apply visible;
361361
}
362362
363-
:deep(.p-togglebutton:hover) .status-indicator {
364-
@apply hidden;
365-
}
366-
367-
:deep(.p-togglebutton) .close-button {
368-
@apply invisible;
369-
}
370-
371363
:deep(.p-scrollpanel-content) {
372364
@apply h-full;
373365
}

0 commit comments

Comments
 (0)