Skip to content

Commit 4ab1e82

Browse files
christian-byrnegithub-actions
andauthored
hide minimap on mobile by default (#6579)
If below large breakpoint, hide the minimap by default. If the user has set the setting before (or closed minimap), their preference will override this. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-6579-hide-minimap-on-mobile-by-default-2a16d73d365081db9c66d209ee046097) by [Unito](https://www.unito.io) --------- Co-authored-by: github-actions <[email protected]>
1 parent adecd25 commit 4ab1e82

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed
-1.79 KB
Loading

src/platform/settings/constants/coreSettings.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import type { ColorPalettes } from '@/schemas/colorPaletteSchema'
66
import type { Keybinding } from '@/schemas/keyBindingSchema'
77
import { NodeBadgeMode } from '@/types/nodeSource'
88
import { LinkReleaseTriggerAction } from '@/types/searchBoxTypes'
9+
import { breakpointsTailwind } from '@vueuse/core'
910

1011
/**
1112
* Core settings are essential configuration parameters required for ComfyUI's basic functionality.
@@ -951,7 +952,7 @@ export const CORE_SETTINGS: SettingParams[] = [
951952
id: 'Comfy.Minimap.Visible',
952953
name: 'Display minimap on canvas',
953954
type: 'hidden',
954-
defaultValue: true,
955+
defaultValue: window.innerWidth >= breakpointsTailwind.lg,
955956
versionAdded: '1.25.0'
956957
},
957958
{

0 commit comments

Comments
 (0)