Skip to content

Commit bdf94bd

Browse files
fix: keep topbar badges in main topbar regardless of workflow tabs position (#6592)
## Summary Fixes incorrect badge placement from PR #6515. Badges should remain in the main topbar row at all times, not move to the second row with workflow tabs. ## Problem PR #6515 added `TopbarBadges` to `SecondRowWorkflowTabs.vue`, causing badges to appear in the second row when workflow tabs position was set to `'Topbar (2nd-row)'`. The original issue was that badges weren't visible when tabs were in second-row mode because they were conditionally rendered only when `workflowTabsPosition === 'Topbar'`. ## Changes - Remove `TopbarBadges` from `SecondRowWorkflowTabs.vue` (badges should never be in second row) - Move `TopbarBadges` from conditional workflow tabs row to main topbar in `TopMenubar.vue` - Badges now always display in main topbar regardless of workflow tabs position ## Testing - Badges visible in main topbar when `workflowTabsPosition === 'Topbar'` - Badges visible in main topbar when `workflowTabsPosition === 'Topbar (2nd-row)'` - Workflow tabs correctly move to second row without badges Fixes issue introduced in #6515
1 parent a83b376 commit bdf94bd

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/components/topbar/SecondRowWorkflowTabs.vue

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
11
<template>
2-
<div class="flex h-full w-auto max-w-full items-center">
2+
<div class="w-auto max-w-full">
33
<WorkflowTabs />
4-
<TopbarBadges />
54
</div>
65
</template>
76

87
<script setup lang="ts">
9-
import TopbarBadges from '@/components/topbar/TopbarBadges.vue'
108
import WorkflowTabs from '@/components/topbar/WorkflowTabs.vue'
119
</script>
1210

src/components/topbar/TopMenubar.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
style="background: var(--border-color)"
77
>
88
<WorkflowTabs />
9-
<TopbarBadges />
109
</div>
1110
<div
1211
v-show="showTopMenu"
@@ -20,6 +19,7 @@
2019
ref="menuRight"
2120
class="comfyui-menu-right flex-shrink-1 overflow-auto"
2221
/>
22+
<TopbarBadges />
2323
<Actionbar />
2424
<CurrentUserButton class="shrink-0" />
2525
</div>

0 commit comments

Comments
 (0)