Skip to content

Conversation

comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Aug 12, 2025

Backport of #4913 to core/1.25

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner August 12, 2025 17:16
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Aug 12, 2025
Copy link

github-actions bot commented Aug 12, 2025

⚠️ Warnings

⚠️ Warning: E2E Test Coverage Missing

If this PR modifies behavior that can be covered by browser-based E2E tests, those tests are required. PRs lacking applicable test coverage may not be reviewed until added. Please add or update browser tests to ensure code quality and prevent regressions.

⚠️ Warning: Visual Documentation Missing

If this PR changes user-facing behavior, visual proof (screen recording or screenshot) is required. PRs without applicable visual documentation may not be reviewed until provided.
You can add it by:

  • GitHub: Drag & drop media directly into the PR description

  • YouTube: Include a link to a short demo

@christian-byrne christian-byrne merged commit ab76669 into core/1.25 Aug 12, 2025
15 checks passed
@christian-byrne christian-byrne deleted the backport-4913-to-1.25 branch August 12, 2025 19:14
@lihaoyun6
Copy link

This doesn't work at all! Tested on a MacBook Pro M1, I can't move the canvas with two fingers; any two-finger gesture is recognized as a mouse wheel.

@christian-byrne
Copy link
Contributor

@webfiltered any idea?

@lihaoyun6
Copy link

lihaoyun6 commented Aug 16, 2025

@webfiltered any idea?

image

By checking the code, I found that this feature is missing an important description: "You need to set the Navigation Mode to [Standard (New)] in ComfyUI settings to use this feature." But the default setting used to be "Legacy"😂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backporting a PR onto a release candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants