Add tooltip to preview disabled nodes#11048
Conversation
mmisol
left a comment
There was a problem hiding this comment.
I'll approve, but I guess at some point we should change our approach for marking a node as preview-disabled. This is not the first issue we see regarding that semi transparent box added on top, preventing normal node usage. It probably makes more sense to simply change the colour of the node instead. Copying @mjkkirschner @aparajit-pratap @Amoursol to get some feedback.
|
This was the problem submitted in the Problem Report, but I agree @mmisol that rather than bespoke solutions we should simply enable all node interactions when the Preview State is turned off with the exception of showing the Geometry in the background preview. |
|
I'm not quite sure I understood why we needed to add another UI element for the tooltip when we already have it. Like @mmisol says, it should be a matter of keeping the existing tooltip enabled even in the preview-off state. |
|
Well, the existing tooltip element was part of the rectangle, and disabling the preview adds a border on top of the node with its bg color set to light blue, and that did not have the tooltip element, which I added. I don't know if there is a way to create a separate tooltip element and bind it to both the rectangle and border. |
|
We can investigate these kind of issues in our Node Refresh work that is currently being explored by UX :) |
Purpose
DYN-3074
Add description, out-ports, in-ports tooltip to nodes which had their preview disabled.
Declarations
Check these if you believe they are true
Reviewers
@DynamoDS/dynamo