From 6081404abb164088f757114c36a07a4c160fcf08 Mon Sep 17 00:00:00 2001 From: Terry Jia Date: Mon, 8 Dec 2025 13:53:35 -0500 Subject: [PATCH] feat: Don't hide scrollbar in BaseModalLayout (#7226) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary - Add showScrollbar prop to BaseModalLayout component to control scrollbar visibility - Enable scrollbar in Templates dialog for better navigation when content overflows The original implementation used scrollbar-hide class by default. Since the intent behind hiding the scrollbar wasn't documented in the original PR (#5290), this change preserves the default behavior (showScrollbar: false) while allowing individual dialogs to opt-in to visible scrollbars. The Templates dialog specifically benefits from a visible scrollbar as it contains a large grid of template cards that requires scrolling. fix https://github.com/Comfy-Org/ComfyUI_frontend/issues/6915 ## Screenshots (if applicable) image ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7226-feat-add-showScrollbar-prop-to-BaseModalLayout-2c36d73d365081b1b55bdcd50ca89030) by [Unito](https://www.unito.io) --- src/components/widget/layout/BaseModalLayout.vue | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/components/widget/layout/BaseModalLayout.vue b/src/components/widget/layout/BaseModalLayout.vue index 779cf3db6d..8057502b84 100644 --- a/src/components/widget/layout/BaseModalLayout.vue +++ b/src/components/widget/layout/BaseModalLayout.vue @@ -162,7 +162,7 @@ const rightAreaClasses = computed(() => { }) const contentContainerClasses = computed(() => { - return cn('min-h-0 px-6 pt-0 pb-10', 'overflow-y-auto scrollbar-hide') + return cn('min-h-0 px-6 pt-0 pb-10', 'overflow-y-auto') }) const rightPanelClasses = computed(() => {