-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for node/input/output tooltips #287
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ref: backend pr comfyanonymous/ComfyUI#3842 |
This looks solid. Thank you for moving the tooltip meta field(s). Also thank you for a node tooltip. I feel the tooltip timeout might be better suited to being adjustable. 500 feels too short (for me) and I like to use 750+ -- seems like an easy setting expose. thank you for the update and changes! |
I'm currently testing and I'm not seeing anything appear on any node. Even after restarting the browser (currently firefox) and making sure that the node I was testing was a newly created one.
Am I doing it wrong? |
Vue implementation of node tooltips