Skip to content

Conversation

DrJKL
Copy link
Contributor

@DrJKL DrJKL commented Aug 24, 2025

See #4109

Credit to @melMass. Just made this new one since the code reorganization would make the merge more painful. Also put the description typing on the LGraphNode instead of in the augments since we have that power now.

Summary

Allows Frontend only nodes to declare a description on the Node class to display to a user.

Changes

  • What: If a node creator has a Frontend only (JS) node and sets a description string, that's displayed instead of the default Frontend only node for ${name}

Review Focus

Screenshots (if applicable)

Did a local tweak using an rgthree node.

image image

┆Issue is synchronized with this Notion page by Unito

@DrJKL DrJKL requested a review from a team as a code owner August 24, 2025 03:46
Copy link

github-actions bot commented Aug 24, 2025

🎭 Playwright Test Results

All tests passed across all browsers!

⏰ Completed at: 08/24/2025, 05:12:35 AM UTC

📊 Test Reports by Browser


🎉 Your tests are passing across all browsers!

@DrJKL DrJKL enabled auto-merge (squash) August 24, 2025 04:59
@DrJKL DrJKL requested a review from AustinMroz August 24, 2025 06:02
@DrJKL DrJKL merged commit 6bf430b into main Aug 24, 2025
12 checks passed
@DrJKL DrJKL deleted the drjkl/redo/4109 branch August 24, 2025 17:11
@christian-byrne christian-byrne mentioned this pull request Aug 27, 2025
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.

3 participants