Skip to content

Conversation

@melMass
Copy link

@melMass melMass commented Jun 8, 2025

I'm not sure if it's enough as I did not test it yet but I think we should be able to set custom description for frontend nodes.

image

┆Issue is synchronized with this Notion page by Unito

@melMass melMass requested a review from a team as a code owner June 8, 2025 16:41
Copy link
Contributor

@webfiltered webfiltered left a comment

Choose a reason for hiding this comment

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

Hrm, could be a little bit more complicated. The description declared in the augmentations file is for the instance, but the description being called in app.ts is actually a static class prop.

We can push this up to litegraph, but it would be good to have this come from the python side, originally. I thought we already had a way to do this, but I might just be thinking of hte tooltips.

@melMass
Copy link
Author

melMass commented Jun 14, 2025

Oh right that's static. Where would it be best?

I think you got confused about one thing, this is for frontend only nodes, python nodes do work like you described already. But FE ones get overridden by the text in the screenshot

@webfiltered
Copy link
Contributor

webfiltered commented Jun 14, 2025

Haha, I thought you were adding something completely custom, outside this repo.. disregard!

Anyway, can you point me at where you're declaring your node, if it's published? Or something similar? Might be better for me to look at this in reverse. (edit: Will fall back to the rgthree one, otherwise)

@melMass
Copy link
Author

melMass commented Jun 14, 2025

Anyway, can you point me at where you're declaring your node, if it's published? Or something similar? Might be better for me to look at this in reverse. (edit: Will fall back to the rgthree one, otherwise)

Yep Note+ in mtb

@DrJKL
Copy link
Contributor

DrJKL commented Aug 24, 2025

Closing in favor of #5187
@melMass 🫡

@DrJKL DrJKL closed this Aug 24, 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