diff --git a/src/components/graph/GraphCanvas.vue b/src/components/graph/GraphCanvas.vue index 27d6c9f0e..a7b26c816 100644 --- a/src/components/graph/GraphCanvas.vue +++ b/src/components/graph/GraphCanvas.vue @@ -179,12 +179,7 @@ const selectionToolboxEnabled = computed(() => ) // Feature flags -const { - shouldRenderVueNodes, - // isViewportCullingEnabled, // Unused - // cullingMargin: featureCullingMargin, // Unused - isDevModeEnabled -} = useFeatureFlags() +const { shouldRenderVueNodes, isDevModeEnabled } = useFeatureFlags() // TransformPane enabled when Vue nodes are enabled OR debug override const debugOverrideVueNodes = ref(true) // Default to true for development @@ -322,8 +317,6 @@ watch( // Transform state for viewport culling const { syncWithCanvas } = useTransformState() -// const cullingMargin = 0.2 // 20% margin outside viewport (unused) - // Replace problematic computed property with proper reactive system const nodesToRender = computed(() => { // Access performanceMetrics to trigger on RAF updates diff --git a/src/composables/useFeatureFlags.ts b/src/composables/useFeatureFlags.ts index 3f1f293a4..b0b129f70 100644 --- a/src/composables/useFeatureFlags.ts +++ b/src/composables/useFeatureFlags.ts @@ -33,18 +33,6 @@ export const useFeatureFlags = () => { } }) - /** - * Enable viewport culling for performance - * When disabled, all Vue nodes render regardless of viewport - */ - const isViewportCullingEnabled = computed(() => { - try { - return settingStore.get('Comfy.VueNodes.ViewportCulling' as any) ?? true - } catch { - return true - } - }) - /** * Development mode features (debug panel, etc.) * Automatically enabled in development builds @@ -71,23 +59,10 @@ export const useFeatureFlags = () => { true ) - /** - * Get culling margin setting - */ - const cullingMargin = computed(() => { - try { - return settingStore.get('Comfy.VueNodes.CullingMargin' as any) ?? 0.2 - } catch { - return 0.2 - } - }) - return { isVueNodesEnabled, isVueWidgetsEnabled, - isViewportCullingEnabled, isDevModeEnabled, - shouldRenderVueNodes, - cullingMargin + shouldRenderVueNodes } } diff --git a/src/constants/coreSettings.ts b/src/constants/coreSettings.ts index b6989d6ae..1c41e6cfb 100644 --- a/src/constants/coreSettings.ts +++ b/src/constants/coreSettings.ts @@ -887,30 +887,5 @@ export const CORE_SETTINGS: SettingParams[] = [ tooltip: 'Render widgets as Vue components within Vue nodes.', type: 'boolean', defaultValue: true - }, - { - id: 'Comfy.VueNodes.ViewportCulling' as any, - category: ['Comfy', 'Vue Nodes', 'Performance'], - experimental: true, - name: 'Enable viewport culling', - tooltip: - 'Only render Vue nodes visible in viewport for better performance.', - type: 'boolean', - defaultValue: true - }, - { - id: 'Comfy.VueNodes.CullingMargin' as any, - category: ['Comfy', 'Vue Nodes', 'Performance'], - experimental: true, - name: 'Viewport culling margin', - tooltip: - 'Percentage of viewport to extend culling bounds (0.2 = 20% margin).', - type: 'slider', - defaultValue: 0.2, - attrs: { - min: 0, - max: 1, - step: 0.05 - } } ]