mirror of
https://github.com/Comfy-Org/ComfyUI_frontend.git
synced 2026-02-23 16:24:06 +00:00
refactor: update outdated tooltip on menu setting (#7330)
Followup from https://github.com/Comfy-Org/ComfyUI_frontend/pull/4312: remove tooltip with outdated info discussing the bottom menu. The bottom menu setting is removed and even if it was not removed, the logic that forced the menu to the top on mobile was removed, so this tooltip is outdated and gives wrong info. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7330-refactor-update-outdated-tooltip-on-menu-setting-2c66d73d36508128b356c6f985d5b12b) by [Unito](https://www.unito.io)
This commit is contained in:
@@ -351,7 +351,7 @@
|
||||
},
|
||||
"Comfy_UseNewMenu": {
|
||||
"name": "Use new menu",
|
||||
"tooltip": "Menu bar position. On mobile devices, the menu is always shown at the top.",
|
||||
"tooltip": "Enable the redesigned top menu bar.",
|
||||
"options": {
|
||||
"Disabled": "Disabled",
|
||||
"Top": "Top"
|
||||
@@ -440,4 +440,4 @@
|
||||
"pysssss_SnapToGrid": {
|
||||
"name": "Always snap to grid"
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -562,8 +562,7 @@ export const CORE_SETTINGS: SettingParams[] = [
|
||||
name: 'Use new menu',
|
||||
type: 'combo',
|
||||
options: ['Disabled', 'Top'],
|
||||
tooltip:
|
||||
'Menu bar position. On mobile devices, the menu is always shown at the top.',
|
||||
tooltip: 'Enable the redesigned top menu bar.',
|
||||
migrateDeprecatedValue: (value: string) => {
|
||||
// Floating is now supported by dragging the docked actionbar off.
|
||||
if (value === 'Floating') {
|
||||
|
||||
Reference in New Issue
Block a user