Page MenuHome

Node Editor: Some nodes lack red line when muted
Closed, DuplicatePublic

Description

Node Editor: Some nodes lack red line when muted

System Information
System: Windows 7 64bit
RAM: 4096 MB
renderer: 'GeForce GTX 760/PCIe/SSE2'
vendor: 'NVIDIA Corporation'
version: '4.5.0 NVIDIA 364.72'

Blender Version
Broken: blender-2.80.0-git.0f5b53ba4dc-windows64

Short description of error
Node Editor: Some nodes lack red line when muted

Exact steps for others to reproduce the error

  1. Open the node editor
  2. Add nodes from the screenshots
  3. Press "M" to mute them
  4. Some nodes have the red crossed through line, others don't

Expected:

  1. Should be consistent (even if it's not a bug)
  2. I can't tell which node is muted and which ones are not

I didn't find any indication in the properties either,
and the changed colors of the node header don't tell anything

Details

Type
Bug

Event Timeline

The red line indicates which input will be passed on at which output when the node is muted. On some nodes that does not make any sense. E.g. passing on the vector as color in the gradient texture node makes no sense.
Nevertheless I agree that some other indication is necessary here (or it should just not be possible to mute those nodes!).

@Brecht Van Lommel (brecht), @William Reynish (billreynish) what do you think?

I think it's best to be consistent. Even if there's only one input and one output, we could still display the red line. The line both helps communicate what happens when a node is disabled, and the fact that it is disabled at all.

I see no immediate downside to just always drawing the red line when disabled. You then also know which socket to use when connecting to a disabled node.

Brecht Van Lommel (brecht) triaged this task as Confirmed, Medium priority.Dec 3 2018, 3:44 PM

We could also make the node header properly red and grey out all the buttons in the node.

If we want to draw the red line, I guess it would have to be connected to some arbitrary point on the left side of the node. Some disabled nodes don't even have input sockets.

Mainly I think the regression compared to 2.7x is the theming, which does not make the header red enough. This should at least be fixed.

Hm, true. Probably better then to just grey out the contents of the node.

The line I see we can't always use, and the theme thing will never be clear enough I don't think.