Skip to content

fix shortcuts explain in README.md#14505

Open
HarangPanul wants to merge 1 commit into
Comfy-Org:masterfrom
HarangPanul:master
Open

fix shortcuts explain in README.md#14505
HarangPanul wants to merge 1 commit into
Comfy-Org:masterfrom
HarangPanul:master

Conversation

@HarangPanul

Copy link
Copy Markdown

fix shortcuts paragraph in README.md

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The shortcuts table in the "Shortcuts" section of README.md was reformatted and consolidated. The duplicate Shift + Drag row that described moving multiple wires was removed, and its behavior was merged into the existing Shift + Drag row. Keybind label formatting was adjusted for consistency, including inline code styling for components such as LMB and Vertical drag.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: fixing the shortcuts explanation in README.md, which aligns with the changeset modifications to the shortcuts table.
Description check ✅ Passed The description is related to the changeset, mentioning the shortcuts paragraph fix in README.md, though minimal in detail.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 162: The inline-code markup for the Alt + C keybind in the README.md
table is malformed with improperly paired backticks. Replace the current `Alt `+
`C` text with properly formatted backticks around each key, using the style
`Alt` + `C` to ensure consistent rendering in the table and match the formatting
convention used elsewhere in the document.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8af2d81e-e5c6-4642-9380-0484feb14e1e

📥 Commits

Reviewing files that changed from the base of the PR and between 90eeeb2 and d50d8fc.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
| `Ctrl` + `S` | Save workflow |
| `Ctrl` + `O` | Load workflow |
| `Ctrl` + `A` | Select all nodes |
| `Alt `+ `C` | Collapse/uncollapse selected nodes |

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Fix the malformed Alt + C formatting.

The current inline-code markup is broken here, so the keybind may render oddly in the table. Please normalize it to the same style used elsewhere, e.g. `Alt` + `C`.

🛠️ Suggested fix
-| Alt `+ `C`                                 | Collapse/uncollapse selected nodes                                                                                 |
+| `Alt` + `C`                                 | Collapse/uncollapse selected nodes                                                                                 |
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| `Alt `+ `C` | Collapse/uncollapse selected nodes |
| `Alt` + `C` | Collapse/uncollapse selected nodes |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` at line 162, The inline-code markup for the Alt + C keybind in the
README.md table is malformed with improperly paired backticks. Replace the
current `Alt `+ `C` text with properly formatted backticks around each key,
using the style `Alt` + `C` to ensure consistent rendering in the table and
match the formatting convention used elsewhere in the document.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant